Always select best favicon bitmap

components/sync_sessions/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 "components/sync_sessions/favicon_cache.h"
 
 #include <utility>
 
 #include "base/location.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 415 matching lines @@
     }
   }
 
   DVLOG(1) << "Triggering favicon load for url " << page_url.spec();
 
   if (!favicon_service_) {
     page_task_map_[page_url] = 0;  // For testing only.
     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(
-          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());
   if (!favicon_url.is_valid() ||
@@ skipping 587 matching lines @@
     favicon_images_sync_processor_->ProcessSyncChanges(FROM_HERE,
                                                        image_deletions);
   }
   if (favicon_tracking_sync_processor_.get()) {
     favicon_tracking_sync_processor_->ProcessSyncChanges(FROM_HERE,
                                                          tracking_deletions);
   }
 }
 
 }  // namespace sync_sessions