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 9f371b8371355a456ca878110a26e4b9b4c6a813..be5d1dc84502eeebf91c0bc5a75b2b3afb166177 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 |
@@ -115,6 +115,11 @@ public class AppBannerManager implements AppBannerView.Observer, AppDetailsDeleg |
nativeReplaceWebContents(mNativePointer, mTab.getWebContents()); |
} |
+ @CalledByNative |
+ private int getPreferredIconSize() { |
+ return AppBannerView.getIconSize(mContentViewCore.getContext()); |
+ } |
+ |
/** |
* Grabs package information for the banner asynchronously. |
* @param url URL for the page that is triggering the banner. |
@@ -127,7 +132,7 @@ public class AppBannerManager implements AppBannerView.Observer, AppDetailsDeleg |
if (sAppDetailsDelegate == null || !isBannerForCurrentPage(url)) return; |
- int iconSize = AppBannerView.getIconSize(mContentViewCore.getContext()); |
+ int iconSize = getPreferredIconSize(); |
sAppDetailsDelegate.getAppDetailsAsynchronously(this, url, packageName, iconSize); |
} |
@@ -225,8 +230,8 @@ public class AppBannerManager implements AppBannerView.Observer, AppDetailsDeleg |
* @return True if the user is still on the same page. |
*/ |
private boolean isBannerForCurrentPage(String bannerUrl) { |
- return mContentViewCore != null && |
- TextUtils.equals(mContentViewCore.getWebContents().getUrl(), bannerUrl); |
+ return mContentViewCore != null |
+ && TextUtils.equals(mContentViewCore.getWebContents().getUrl(), bannerUrl); |
} |
private static native boolean nativeIsEnabled(); |