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

chrome/browser/banners/app_banner_manager_desktop.cc

Index: chrome/browser/banners/app_banner_manager_desktop.cc
diff --git a/chrome/browser/banners/app_banner_manager_desktop.cc b/chrome/browser/banners/app_banner_manager_desktop.cc
index 09d7fdf88f27a7dcc3338930cc277e878871fcd7..b587d20ce3a468089d37cb06bc54f54bb7eccec3 100644
--- a/chrome/browser/banners/app_banner_manager_desktop.cc
+++ b/chrome/browser/banners/app_banner_manager_desktop.cc
@@ -10,7 +10,7 @@
 #include "extensions/common/constants.h"
 
 namespace {
-// TODO(dominickn) Enforce the set of icons which will guarantee the best
+// TODO(dominickn) Ientify the best minimum icon size to guarantee the best

[dominickn, 2015/09/14 00:07:09]

Nit: "Identify"

[Lalit Maganti, 2015/09/14 11:35:19]

Done.

 // user experience.
 int kMinimumIconSize = extension_misc::EXTENSION_ICON_LARGE;
 }  // anonymous namespace
@@ -25,15 +25,15 @@ bool AppBannerManagerDesktop::IsEnabled() {
 }
 
 AppBannerDataFetcher* AppBannerManagerDesktop::CreateAppBannerDataFetcher(
-    base::WeakPtr<AppBannerDataFetcher::Delegate> weak_delegate,
-    const int ideal_icon_size) {
+    base::WeakPtr<AppBannerDataFetcher::Delegate> weak_delegate) {
   return new AppBannerDataFetcherDesktop(web_contents(), weak_delegate,
-                                         ideal_icon_size);
+                                         kMinimumIconSize,
+                                         kMinimumIconSize);
 }
 
 AppBannerManagerDesktop::AppBannerManagerDesktop(
     content::WebContents* web_contents)
-    : AppBannerManager(web_contents, kMinimumIconSize) {
+    : AppBannerManager(web_contents) {
 }
 
 }  // namespace banners