webapps: allow callers of icon downloader/selector to specify a minimum size

chrome/browser/android/banners/app_banner_manager_android.h

Index: chrome/browser/android/banners/app_banner_manager_android.h
diff --git a/chrome/browser/android/banners/app_banner_manager_android.h b/chrome/browser/android/banners/app_banner_manager_android.h
index 4fd358b9a42891e9cc15bf184e8b745c13772bbb..7fc85facb0d8a9de8ead242df8bf860a5b0e73a8 100644
--- a/chrome/browser/android/banners/app_banner_manager_android.h
+++ b/chrome/browser/android/banners/app_banner_manager_android.h
@@ -26,9 +26,7 @@ class AppBannerManagerAndroid : public AppBannerManager {
   static bool Register(JNIEnv* env);
 
   AppBannerManagerAndroid(JNIEnv* env,
-                          jobject obj,
-                          int ideal_splash_image_size_in_dp,
-                          int ideal_icon_size_in_dp);
+                          jobject obj);

[gone, 2015/09/15 18:46:07]

nit: put back on one line?

[Lalit Maganti, 2015/09/15 18:48:33]

Done.

   ~AppBannerManagerAndroid() override;
 
   // Destroys the AppBannerManagerAndroid.
@@ -53,8 +51,7 @@ class AppBannerManagerAndroid : public AppBannerManager {
 
  protected:
   AppBannerDataFetcher* CreateAppBannerDataFetcher(
-      base::WeakPtr<AppBannerDataFetcher::Delegate> weak_delegate,
-      const int ideal_icon_size_in_dp) override;
+      base::WeakPtr<AppBannerDataFetcher::Delegate> weak_delegate) override;
 
  private:
   // AppBannerDataFetcher::Delegate overrides.
@@ -69,9 +66,6 @@ class AppBannerManagerAndroid : public AppBannerManager {
   std::string ExtractQueryValueForName(const GURL& url,
                                        const std::string& name);
 
-  // Ideal splash screen image size to use.
-  const int ideal_splash_image_size_in_dp_;
-
   // AppBannerManager on the Java side.
   JavaObjectWeakGlobalRef weak_java_banner_view_manager_;