Index: chrome/android/java/src/org/chromium/chrome/browser/Tab.java |
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 7ba426f4e0c1f3ff804701a5d557617c5f84d8cb..33f1b73097c33afd0d785ac7a352ae41a4d89e62 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/Tab.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/Tab.java |
@@ -32,7 +32,6 @@ import org.chromium.content.browser.ContentViewCore; |
import org.chromium.content.browser.LoadUrlParams; |
import org.chromium.content.browser.NavigationClient; |
import org.chromium.content.browser.NavigationHistory; |
-import org.chromium.content.browser.PageInfo; |
import org.chromium.content.browser.WebContentsObserverAndroid; |
import org.chromium.content_public.browser.WebContents; |
import org.chromium.ui.base.Clipboard; |
@@ -453,8 +452,7 @@ public class Tab implements NavigationClient { |
* be {@code null}, if the tab is frozen or being initialized or destroyed. |
*/ |
public View getView() { |
- PageInfo pageInfo = getPageInfo(); |
- return pageInfo != null ? pageInfo.getView() : null; |
+ return mNativePage != null ? mNativePage.getView() : mContentView; |
} |
/** |
@@ -535,7 +533,9 @@ public class Tab implements NavigationClient { |
* @return The background color of the tab. |
*/ |
public int getBackgroundColor() { |
- return getPageInfo() != null ? getPageInfo().getBackgroundColor() : Color.WHITE; |
+ if (mNativePage != null) return mNativePage.getBackgroundColor(); |
+ if (mContentViewCore != null) return mContentViewCore.getBackgroundColor(); |
+ return Color.WHITE; |
} |
/** |
@@ -590,14 +590,6 @@ public class Tab implements NavigationClient { |
} |
/** |
- * @return A {@link PageInfo} describing the current page. This is always not {@code null} |
- * except during initialization, destruction, and when the tab is frozen. |
- */ |
- public PageInfo getPageInfo() { |
- return mNativePage != null ? mNativePage : mContentView; |
- } |
- |
- /** |
* @return The {@link NativePage} associated with the current page, or {@code null} if there is |
* no current page or the current page is displayed using something besides |
* {@link NativePage}. |
@@ -768,7 +760,7 @@ public class Tab implements NavigationClient { |
/** |
* Completes the {@link ContentView} specific initialization around a native WebContents |
- * pointer. {@link #getPageInfo()} will still return the {@link NativePage} if there is one. |
+ * pointer. {@link #getNativePage()} will still return the {@link NativePage} if there is one. |
* All initialization that needs to reoccur after a web contents swap should be added here. |
* <p /> |
* NOTE: If you attempt to pass a native WebContents that does not have the same incognito |
@@ -868,7 +860,9 @@ public class Tab implements NavigationClient { |
*/ |
@CalledByNative |
public String getTitle() { |
- return getPageInfo() != null ? getPageInfo().getTitle() : ""; |
+ if (mNativePage != null) return mNativePage.getTitle(); |
+ if (mContentViewCore != null) return mContentViewCore.getTitle(); |
+ return ""; |
} |
/** |
@@ -929,7 +923,7 @@ public class Tab implements NavigationClient { |
private void destroyNativePageInternal(NativePage nativePage) { |
if (nativePage == null) return; |
- assert getPageInfo() != nativePage : "Attempting to destroy active page."; |
+ assert nativePage != mNativePage : "Attempting to destroy active page."; |
nativePage.destroy(); |
} |