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

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

Issue 331163003: [Refactor] Change FaviconService API to take in a desired pixel size (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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 715998fa0614a73b8410bc17bbb4d374955d6f0c..c63b8bb79452d5f4d0e45ad1ddb4e815ed93bffb 100644
--- a/chrome/browser/ui/cocoa/history_menu_bridge.mm
+++ b/chrome/browser/ui/cocoa/history_menu_bridge.mm
@@ -32,7 +32,6 @@
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
#include "ui/gfx/codec/png_codec.h"
-#include "ui/gfx/favicon_size.h"
#include "ui/gfx/image/image.h"
namespace {
@@ -457,8 +456,7 @@ void HistoryMenuBridge::GetFaviconForHistoryItem(HistoryItem* item) {
FaviconServiceFactory::GetForProfile(profile_, Profile::EXPLICIT_ACCESS);
base::CancelableTaskTracker::TaskId task_id =
service->GetFaviconImageForPageURL(
- FaviconService::FaviconForPageURLParams(
- item->url, favicon_base::FAVICON, gfx::kFaviconSize),
+ item->url,
base::Bind(
&HistoryMenuBridge::GotFaviconData, base::Unretained(this), item),
&cancelable_task_tracker_);
« no previous file with comments | « chrome/browser/ui/android/navigation_popup.cc ('k') | chrome/browser/ui/search_engines/template_url_table_model.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698