Find Favicon in priority of icon_type.

chrome/browser/favicon/favicon_service.cc

 // Copyright (c) 2012 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/favicon/favicon_service.h"
 
 #include "base/hash.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "chrome/browser/favicon/favicon_util.h"
 #include "chrome/browser/history/history_backend.h"
@@ skipping 27 matching lines @@
 // service.
 CancelableTaskTracker::TaskId RunWithEmptyResultAsync(
     const FaviconService::FaviconResultsCallback& callback,
     CancelableTaskTracker* tracker) {
   return tracker->PostTask(
       base::MessageLoopProxy::current().get(),
       FROM_HERE,
       Bind(callback, std::vector<chrome::FaviconBitmapResult>()));
 }
 
+// Return the TaskId of the one to retreive the favicon from chrome specific

[pkotwicz, 2013/10/15 21:39:41]

How about: "Returns the TaskId to retrieve the favicon from a chrome-specific
URL."

[michaelbai, 2013/10/15 22:52:34]

Done.

+// URL.
+CancelableTaskTracker::TaskId GetFaviconForChromeURL(
+    Profile* profile,
+    const GURL& page_url,
+    const std::vector<ui::ScaleFactor>& desired_scale_factors,
+    const FaviconService::FaviconResultsCallback& callback,
+    CancelableTaskTracker* tracker) {
+  CancelableTaskTracker::IsCanceledCallback is_canceled_cb;
+  CancelableTaskTracker::TaskId id = tracker->NewTrackedTaskId(&is_canceled_cb);
+  FaviconService::FaviconResultsCallback cancelable_cb =
+      Bind(&CancelOrRunFaviconResultsCallback, is_canceled_cb, callback);
+  ChromeWebUIControllerFactory::GetInstance()->GetFaviconForURL(profile,
+      page_url, desired_scale_factors, cancelable_cb);
+  return id;
+}
+
 }  // namespace
 
 FaviconService::FaviconService(HistoryService* history_service)
     : history_service_(history_service) {
 }
 
 // static
 void FaviconService::FaviconResultsCallbackRunner(
     const FaviconResultsCallback& callback,
     const std::vector<chrome::FaviconBitmapResult>* results) {
@@ skipping 104 matching lines @@
       params,
       desired_scale_factors,
       Bind(&FaviconService::RunFaviconRawCallbackWithBitmapResults,
            base::Unretained(this),
            callback,
            params.desired_size_in_dip,
            desired_scale_factor),
       tracker);
 }
 
+CancelableTaskTracker::TaskId FaviconService::GetLargestRawFaviconForURL(
+    Profile* profile,
+    const GURL& page_url,
+    const std::vector<int>& icon_types,
+    int minimal_size_in_pixels,
+    const FaviconRawCallback& callback,
+    CancelableTaskTracker* tracker) {
+  return GetLargestRawFaviconForURLImpl(
+      profile, page_url, icon_types, minimal_size_in_pixels,
+      Bind(&FaviconService::RunFaviconRawCallbackWithBitmapResults,
+           base::Unretained(this),
+           callback,
+           0,
+           ui::ScaleFactor()),
+      tracker);
+}
+
 CancelableTaskTracker::TaskId FaviconService::GetFaviconForURL(
     const FaviconForURLParams& params,
     const FaviconResultsCallback& callback,
     CancelableTaskTracker* tracker) {
   return GetFaviconForURLImpl(params,
                               FaviconUtil::GetFaviconScaleFactors(),
                               callback,
                               tracker);
 }
 
@@ skipping 95 matching lines @@
 
 FaviconService::~FaviconService() {}
 
 CancelableTaskTracker::TaskId FaviconService::GetFaviconForURLImpl(
     const FaviconForURLParams& params,
     const std::vector<ui::ScaleFactor>& desired_scale_factors,
     const FaviconResultsCallback& callback,
     CancelableTaskTracker* tracker) {
   if (params.page_url.SchemeIs(chrome::kChromeUIScheme) ||
       params.page_url.SchemeIs(extensions::kExtensionScheme)) {
-    CancelableTaskTracker::IsCanceledCallback is_canceled_cb;
-    CancelableTaskTracker::TaskId id =
-        tracker->NewTrackedTaskId(&is_canceled_cb);
-
-    FaviconResultsCallback cancelable_cb =
-        Bind(&CancelOrRunFaviconResultsCallback, is_canceled_cb, callback);
-    ChromeWebUIControllerFactory::GetInstance()->GetFaviconForURL(
-        params.profile, params.page_url, desired_scale_factors, cancelable_cb);
-    return id;
+    return GetFaviconForChromeURL(params.profile, params.page_url,
+                                  desired_scale_factors, callback, tracker);
   } else if (history_service_) {
     return history_service_->GetFaviconsForURL(params.page_url,
                                                params.icon_types,
                                                params.desired_size_in_dip,
                                                desired_scale_factors,
                                                callback,
                                                tracker);
-  } else {
-    return RunWithEmptyResultAsync(callback, tracker);
   }
+  return RunWithEmptyResultAsync(callback, tracker);
+}
+
+CancelableTaskTracker::TaskId FaviconService::GetLargestRawFaviconForURLImpl(

[pkotwicz, 2013/10/15 21:39:41]

Can you explain why you need both GetLargestRawFaviconForURL() and
GetLargestRawFaviconForURLImpl()

[michaelbai, 2013/10/15 22:52:34]

Removed

On 2013/10/15 21:39:41, pkotwicz wrote:

+    Profile* profile,
+    const GURL& page_url,
+    const std::vector<int>& icon_types,
+    int minimal_size_in_pixels,
+    const FaviconResultsCallback& callback,
+    CancelableTaskTracker* tracker) {
+  if (page_url.SchemeIs(chrome::kChromeUIScheme) ||
+      page_url.SchemeIs(extensions::kExtensionScheme)) {
+    return GetFaviconForChromeURL(profile, page_url,
+        std::vector<ui::ScaleFactor>(), callback, tracker);

[pkotwicz, 2013/10/15 21:39:41]

You should pass in a vector with ui::SCALE_FACTOR_100P. Passing in an empty
vector is wrong given ChromeWebUIControllerFactory::GetFaviconForURL().

[michaelbai, 2013/10/15 22:52:34]

Thanks

+  } else if (history_service_) {
+    return history_service_->GetLargestFaviconForURL(page_url, icon_types,
+        minimal_size_in_pixels, callback, tracker);
+  }
+  return RunWithEmptyResultAsync(callback, tracker);
 }
 
 void FaviconService::RunFaviconImageCallbackWithBitmapResults(
     const FaviconImageCallback& callback,
     int desired_size_in_dip,
     const std::vector<chrome::FaviconBitmapResult>& favicon_bitmap_results) {
   chrome::FaviconImageResult image_result;
   image_result.image = FaviconUtil::SelectFaviconFramesFromPNGs(
       favicon_bitmap_results,
       FaviconUtil::GetFaviconScaleFactors(),
@@ skipping 48 matching lines @@
   if (!gfx::PNGCodec::EncodeBGRASkBitmap(resized_image.AsBitmap(), false,
                                          &resized_bitmap_data)) {
     callback.Run(chrome::FaviconBitmapResult());
     return;
   }
 
   bitmap_result.bitmap_data = base::RefCountedBytes::TakeVector(
       &resized_bitmap_data);
   callback.Run(bitmap_result);
 }