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

Unified Diff: chrome/browser/ui/cocoa/history_menu_bridge.mm

Issue 330603004: Rename FaviconBitmapXxx to FaviconRawBitmapXxx (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix android Created 6 years, 6 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/cocoa/history_menu_bridge.mm
diff --git a/chrome/browser/ui/cocoa/history_menu_bridge.mm b/chrome/browser/ui/cocoa/history_menu_bridge.mm
index 3fd89b72451fd7c7b99ee42e867d4863bd7b5f57..1c20e787f0245097cba64923800d22192ef1c711 100644
--- a/chrome/browser/ui/cocoa/history_menu_bridge.mm
+++ b/chrome/browser/ui/cocoa/history_menu_bridge.mm
@@ -456,12 +456,13 @@ HistoryMenuBridge::HistoryItem* HistoryMenuBridge::HistoryItemForTab(
void HistoryMenuBridge::GetFaviconForHistoryItem(HistoryItem* item) {
FaviconService* service =
FaviconServiceFactory::GetForProfile(profile_, Profile::EXPLICIT_ACCESS);
- base::CancelableTaskTracker::TaskId task_id = service->GetFaviconImageForURL(
- FaviconService::FaviconForURLParams(
- item->url, favicon_base::FAVICON, gfx::kFaviconSize),
- base::Bind(
- &HistoryMenuBridge::GotFaviconData, base::Unretained(this), item),
- &cancelable_task_tracker_);
+ base::CancelableTaskTracker::TaskId task_id =
+ service->GetFaviconImageForPageURL(
+ FaviconService::FaviconForPageURLParams(
+ item->url, favicon_base::FAVICON, gfx::kFaviconSize),
+ base::Bind(
+ &HistoryMenuBridge::GotFaviconData, base::Unretained(this), item),
+ &cancelable_task_tracker_);
item->icon_task_id = task_id;
item->icon_requested = true;
}
« no previous file with comments | « chrome/browser/ui/ash/launcher/launcher_favicon_loader.cc ('k') | chrome/browser/ui/toolbar/back_forward_menu_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698