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

chrome/browser/banners/app_banner_data_fetcher.h

Index: chrome/browser/banners/app_banner_data_fetcher.h
diff --git a/chrome/browser/banners/app_banner_data_fetcher.h b/chrome/browser/banners/app_banner_data_fetcher.h
index 6d06e56bfe0af61b9b4b6c632645c8f509c39a84..94be5d04c27886c3f2e48e6f004ff3c63560a002 100644
--- a/chrome/browser/banners/app_banner_data_fetcher.h
+++ b/chrome/browser/banners/app_banner_data_fetcher.h
@@ -57,7 +57,8 @@ class AppBannerDataFetcher
 
   AppBannerDataFetcher(content::WebContents* web_contents,
                        base::WeakPtr<Delegate> weak_delegate,
-                       int ideal_icon_size_in_dp);
+                       int ideal_icon_size_in_dp,
+                       int minimum_icon_size_in_dp);
 
   // Begins creating a banner for the URL being displayed by the Delegate's
   // WebContents.
@@ -170,6 +171,7 @@ class AppBannerDataFetcher
                                        content::WebContents* web_contents);
 
   const int ideal_icon_size_in_dp_;
+  const int minimum_icon_size_in_dp_;
   const base::WeakPtr<Delegate> weak_delegate_;

[dominickn, 2015/09/11 00:29:58]

Nit: reorder these to put the int members after the weak delegate so the order
of arguments in the constructor matches the order of members in the class.

[Lalit Maganti, 2015/09/11 13:06:00]

Done.

   base::ObserverList<Observer> observer_list_;
   bool is_active_;