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

chrome/browser/android/provider/chrome_browser_provider.cc

Index: chrome/browser/android/provider/chrome_browser_provider.cc
diff --git a/chrome/browser/android/provider/chrome_browser_provider.cc b/chrome/browser/android/provider/chrome_browser_provider.cc
index 2e77c8e9b85bdcd76c8c069362a56f315bacd79f..188c286707137912ffdb89110cfe61e0e8fd3fd2 100644
--- a/chrome/browser/android/provider/chrome_browser_provider.cc
+++ b/chrome/browser/android/provider/chrome_browser_provider.cc
@@ -4,6 +4,7 @@
 
 #include "chrome/browser/android/provider/chrome_browser_provider.h"
 
+#include <cmath>
 #include <list>
 #include <utility>
 
@@ -683,14 +684,13 @@ class BookmarkIconFetchTask : public FaviconServiceTask {
   favicon_base::FaviconRawBitmapResult Run(const GURL& url) {
     float max_scale = ui::GetScaleForScaleFactor(
         ResourceBundle::GetSharedInstance().GetMaxScaleFactor());
+    int desired_size_in_pixel = std::ceil(gfx::kFaviconSize * max_scale);
     RunAsyncRequestOnUIThreadBlocking(
         base::Bind(&FaviconService::GetRawFaviconForPageURL,
                    base::Unretained(service()),
-                   FaviconService::FaviconForPageURLParams(
-                       url,
-                       favicon_base::FAVICON | favicon_base::TOUCH_ICON,
-                       gfx::kFaviconSize),
-                   max_scale,
+                   url,
+                   favicon_base::FAVICON | favicon_base::TOUCH_ICON,
+                   desired_size_in_pixel,
                    base::Bind(&BookmarkIconFetchTask::OnFaviconRetrieved,
                               base::Unretained(this)),
                    cancelable_tracker()));