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

Unified Diff: chrome/browser/ui/toolbar/back_forward_menu_model.h

Issue 10870022: Change FaviconData to be able to return data for multiple bitmaps for same icon URL (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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/toolbar/back_forward_menu_model.h
diff --git a/chrome/browser/ui/toolbar/back_forward_menu_model.h b/chrome/browser/ui/toolbar/back_forward_menu_model.h
index 97f1155295bf8900777fb8cd69b63a151c6da621..23f7a3eaf333e343391de19235f5b23299b1a83a 100644
--- a/chrome/browser/ui/toolbar/back_forward_menu_model.h
+++ b/chrome/browser/ui/toolbar/back_forward_menu_model.h
@@ -94,8 +94,9 @@ class BackForwardMenuModel : public ui::MenuModel {
void FetchFavicon(content::NavigationEntry* entry);
// Callback from the favicon service.
- void OnFavIconDataAvailable(FaviconService::Handle handle,
- history::FaviconData favicon);
+ void OnFavIconDataAvailable(
+ FaviconService::Handle handle,
+ const history::FaviconImageResult& image_result);
// Allows the unit test to use its own dummy tab contents.
void set_test_web_contents(content::WebContents* test_web_contents) {

Powered by Google App Engine
This is Rietveld 408576698