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

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

Issue 2795763004: Treat touch icons just like non-touch icons on mobile
Patch Set: 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_driver_observer.h
diff --git a/components/favicon/core/favicon_driver_observer.h b/components/favicon/core/favicon_driver_observer.h
index ce80877a7376948431f15b4c0741274cb746c15d..13d6840f294e9105296a56291170529ab99e6393 100644
--- a/components/favicon/core/favicon_driver_observer.h
+++ b/components/favicon/core/favicon_driver_observer.h
@@ -27,12 +27,9 @@ class FaviconDriverObserver {
// favicon of type favicon_base::FAVICON. If the page does not provide a
// 16x16 DIP icon, the icon is generated by resizing another icon.
NON_TOUCH_16_DIP,
- // Largest icon specified by the page which is of type
- // favicon_base::FAVICON.
- NON_TOUCH_LARGEST,
- // Largest icon specified by the page which is of type
- // favicon_base::TOUCH_ICON or of type favicon_base::TOUCH_PRECOMPOSED_ICON.
- TOUCH_LARGEST
+ // Largest icon specified by the page, of any type (includes favicons and
+ // touch icons).
+ LARGEST
};
FaviconDriverObserver() {}

Powered by Google App Engine
This is Rietveld 408576698