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

Unified Diff: chrome/browser/ui/metro_pin_tab_helper_win.cc

Issue 12945007: Fix Notifications Icon loading. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: tiny fix Created 7 years, 9 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/ui/metro_pin_tab_helper_win.cc
diff --git a/chrome/browser/ui/metro_pin_tab_helper_win.cc b/chrome/browser/ui/metro_pin_tab_helper_win.cc
index f5078f37d8e66d40849e258adf313e844a137438..fd5dcb7bdbb7b1756ff50a4eb57551affec168da 100644
--- a/chrome/browser/ui/metro_pin_tab_helper_win.cc
+++ b/chrome/browser/ui/metro_pin_tab_helper_win.cc
@@ -400,7 +400,9 @@ void MetroPinTabHelper::TogglePinnedToStartScreen() {
iter != favicon_url_candidates_.end();
++iter) {
favicon_chooser_->AddPendingRequest(
- web_contents()->DownloadFavicon(iter->icon_url, image_size,
+ web_contents()->DownloadFavicon(iter->icon_url,
+ true, // favicon
jam 2013/03/20 02:44:42 ditto
+ image_size,
base::Bind(&MetroPinTabHelper::DidDownloadFavicon,
base::Unretained(this))));
}

Powered by Google App Engine
This is Rietveld 408576698