Index: chrome/android/java/src/org/chromium/chrome/browser/banners/AppBannerManager.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/banners/AppBannerManager.java b/chrome/android/java/src/org/chromium/chrome/browser/banners/AppBannerManager.java |
index cd0a59ae9e74325ce7629ea57e28c20970b9b09b..851e19950f8a422ab0226caf66b705dd69afe36b 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/banners/AppBannerManager.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/banners/AppBannerManager.java |
@@ -40,9 +40,6 @@ public class AppBannerManager extends EmptyTabObserver { |
/** Pointer to the native side AppBannerManager. */ |
private long mNativePointer; |
- /** Tab that the AppBannerView/AppBannerManager is owned by. */ |
- private final Tab mTab; |
- |
/** |
* Checks if app banners are enabled. |
* @return True if banners are enabled, false otherwise. |
@@ -70,19 +67,18 @@ public class AppBannerManager extends EmptyTabObserver { |
*/ |
public AppBannerManager(Tab tab, Context context) { |
mNativePointer = nativeInit(); |
- mTab = tab; |
- updatePointers(); |
+ updatePointers(tab); |
} |
@Override |
public void onWebContentsSwapped(Tab tab, boolean didStartLoad, |
boolean didFinishLoad) { |
- updatePointers(); |
+ updatePointers(tab); |
} |
@Override |
public void onContentChanged(Tab tab) { |
- updatePointers(); |
+ updatePointers(tab); |
} |
/** |
@@ -96,8 +92,8 @@ public class AppBannerManager extends EmptyTabObserver { |
/** |
* Updates which WebContents the native AppBannerManager is monitoring. |
*/ |
- private void updatePointers() { |
- nativeReplaceWebContents(mNativePointer, mTab.getWebContents()); |
+ private void updatePointers(Tab tab) { |
+ nativeReplaceWebContents(mNativePointer, tab.getWebContents()); |
} |
/** |