[Refactor] Change FaviconService API to take in a desired pixel size

chrome/browser/notifications/message_center_settings_controller.cc

Index: chrome/browser/notifications/message_center_settings_controller.cc
diff --git a/chrome/browser/notifications/message_center_settings_controller.cc b/chrome/browser/notifications/message_center_settings_controller.cc
index 69e9ca9a6a39969b1e1c1b12b9e8f40656992793..1278631549abd3b352e4b876e1450e06b3474bc9 100644
--- a/chrome/browser/notifications/message_center_settings_controller.cc
+++ b/chrome/browser/notifications/message_center_settings_controller.cc
@@ -271,15 +271,11 @@ void MessageCenterSettingsController::GetNotifierList(
         name,
         notification_service->IsNotifierEnabled(notifier_id)));
     patterns_[name] = iter->primary_pattern;
-    FaviconService::FaviconForPageURLParams favicon_params(
-        url,
-        favicon_base::FAVICON | favicon_base::TOUCH_ICON,

[sky, 2014/07/10 00:03:39]

Don't you lose the TOUCH_ICON with this change?

[pkotwicz, 2014/07/11 01:23:08]

TOUCH_ICON is android and IOS only and based on build/common.gypi neither uses
the message center.

-        message_center::kSettingsIconSize);
     // Note that favicon service obtains the favicon from history. This means
     // that it will fail to obtain the image if there are no history data for
     // that URL.
     favicon_service->GetFaviconImageForPageURL(
-        favicon_params,
+        url,
         base::Bind(&MessageCenterSettingsController::OnFaviconLoaded,
                    base::Unretained(this),
                    url),