Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(297)

Unified Diff: components/favicon/core/favicon_service.h

Issue 2823093002: Make FaviconService::GetRawFaviconForPageURL() select the best candidate among all the icon types (Closed)
Patch Set: Merge branch 'icon_type0' into icon_type Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/favicon/core/favicon_service.h
diff --git a/components/favicon/core/favicon_service.h b/components/favicon/core/favicon_service.h
index 526b5f613d139741210d271623102a91445ea9f5..0ad17be7947a7c4aed9541e564aab72b3e841cb5 100644
--- a/components/favicon/core/favicon_service.h
+++ b/components/favicon/core/favicon_service.h
@@ -88,13 +88,9 @@ class FaviconService : public KeyedService {
// Requests the favicon for the page at |page_url| with one of |icon_types|
// and with |desired_size_in_pixel|. |icon_types| can be any combination of
- // IconTypes. If favicon bitmaps for several IconTypes are available, the
- // favicon bitmap is chosen in the priority of TOUCH_PRECOMPOSED_ICON,
- // TOUCH_ICON and FAVICON. If there is no favicon bitmap of size
- // |desired_size_in_pixel|, the favicon bitmap which best matches
- // |desired_size_in_pixel| is resized. If |desired_size_in_pixel| is 0,
- // the largest favicon bitmap is returned. Results with a higher priority
- // IconType are preferred over an exact match of the favicon bitmap size.
+ // IconTypes. If there is no favicon bitmap of size |desired_size_in_pixel|,
+ // the favicon bitmap which best matches |desired_size_in_pixel| is resized.
+ // If |desired_size_in_pixel| is 0, the largest favicon bitmap is returned.
virtual base::CancelableTaskTracker::TaskId GetRawFaviconForPageURL(
const GURL& page_url,
int icon_types,

Powered by Google App Engine
This is Rietveld 408576698