Don't overwrite Tab title if unnecessary

Messed up the logic slightly when refactoring the code. If a
native page exists, the title should always be the one belonging to the native page.

Original logic located here, in Tab.getTitle():
https://codereview.chromium.org/834723002

BUG=450997
TBR=newt
Review URL: https://codereview.chromium.org/871583004

Cr-Commit-Position: refs/heads/master@{#312698}

Review URL: https://codereview.chromium.org/867343002

Cr-Commit-Position: refs/branch-heads/2272@{#97}
Cr-Branched-From: 827a380cfdb31aa54c8d56e63ce2c3fd8c3ba4d4-refs/heads/master@{#310958}

1 files changed, 5 insertions, 2 deletions

diff --git a/chrome/android/java/src/org/chromium/chrome/browser/Tab.java b/chrome/android/java/src/org/chromium/chrome/browser/Tab.java
index 21565a6..6236046 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/Tab.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/Tab.java
@@ -1522,8 +1522,11 @@ public class Tab implements ViewGroup.OnHierarchyChangeListener,
         // When restoring the tabs, the title will no longer be populated, so request it from the
         // ContentViewCore or NativePage (if present).
         String title = "";
-        if (mNativePage != null) title = mNativePage.getTitle();
-        if (getWebContents() != null) title = getWebContents().getTitle();
+        if (mNativePage != null) {
+            title = mNativePage.getTitle();
+        } else if (getWebContents() != null) {
+            title = getWebContents().getTitle();
+        }
         updateTitle(title);
     }