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

chrome/browser/banners/app_banner_manager_desktop.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/banners/app_banner_manager_desktop.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/banners/app_banner_data_fetcher_desktop.h"
 #include "chrome/common/chrome_switches.h"
 #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
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(banners::AppBannerManagerDesktop);
 
 namespace banners {
 
 bool AppBannerManagerDesktop::IsEnabled() {
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableAddToShelf);
 }
 
 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