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

chrome/browser/sync/glue/favicon_cache.cc

 // Copyright 2013 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/sync/glue/favicon_cache.h"
 
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/histogram.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/favicon/favicon_service.h"
@@ skipping 425 matching lines @@
   }
   FaviconService* favicon_service =
       FaviconServiceFactory::GetForProfile(profile_, Profile::EXPLICIT_ACCESS);
   if (!favicon_service)
     return;
   // TODO(zea): This appears to only fetch one favicon (best match based on
   // desired_size_in_dip). Figure out a way to fetch all favicons we support.
   // See crbug.com/181068.
   base::CancelableTaskTracker::TaskId id =
       favicon_service->GetFaviconForPageURL(
-          FaviconService::FaviconForPageURLParams(
-              page_url, SupportedFaviconTypes(), kMaxFaviconResolution),
+          page_url,
+          SupportedFaviconTypes(),
+          kMaxFaviconResolution,
           base::Bind(&FaviconCache::OnFaviconDataAvailable,
                      weak_ptr_factory_.GetWeakPtr(),
                      page_url),
           &cancelable_task_tracker_);
   page_task_map_[page_url] = id;
 }
 
 void FaviconCache::OnFaviconVisited(const GURL& page_url,
                                     const GURL& favicon_url) {
   DCHECK(page_url.is_valid());
@@ skipping 600 matching lines @@
 
 size_t FaviconCache::NumFaviconsForTest() const {
   return synced_favicons_.size();
 }
 
 size_t FaviconCache::NumTasksForTest() const {
   return page_task_map_.size();
 }
 
 }  // namespace browser_sync