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

Unified Diff: chrome/browser/android/tab_android.cc

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: chrome/browser/android/tab_android.cc
diff --git a/chrome/browser/android/tab_android.cc b/chrome/browser/android/tab_android.cc
index 45f18987aa13cd35cd78968a8a37d2c2116ee9b2..ab360f7ccd2c1a800057da7953f3ccecf6f85fd7 100644
--- a/chrome/browser/android/tab_android.cc
+++ b/chrome/browser/android/tab_android.cc
@@ -304,10 +304,8 @@ void TabAndroid::OnFaviconUpdated(favicon::FaviconDriver* favicon_driver,
const GURL& icon_url,
bool icon_url_changed,
const gfx::Image& image) {
- if (notification_icon_type != NON_TOUCH_LARGEST &&
- notification_icon_type != TOUCH_LARGEST) {
+ if (notification_icon_type != LARGEST)
return;
- }
SkBitmap favicon = image.AsImageSkia().GetRepresentation(1.0f).sk_bitmap();
if (favicon.empty())
« no previous file with comments | « no previous file | components/favicon/core/favicon_driver_impl.h » ('j') | components/favicon/core/favicon_handler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698