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

Unified Diff: components/history/core/browser/thumbnail_database.h

Issue 2828173002: [Refactor] Simplify HistoryBackend::UpdateFaviconMappingsAndFetchImpl() signature (Closed)
Patch Set: Merge branch 'master' into icon_type0 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: components/history/core/browser/thumbnail_database.h
diff --git a/components/history/core/browser/thumbnail_database.h b/components/history/core/browser/thumbnail_database.h
index 9f81d41f46daca2f9078e6d7f54a0bd2ea3099dd..b277d0e6cf4bbff032a26f77ef68207f8e72a1e0 100644
--- a/components/history/core/browser/thumbnail_database.h
+++ b/components/history/core/browser/thumbnail_database.h
@@ -129,16 +129,11 @@ class ThumbnailDatabase {
bool SetFaviconOutOfDate(favicon_base::FaviconID icon_id);
// Returns the id of the entry in the favicon database with the specified url
- // and icon type. If |required_icon_type| contains multiple icon types and
- // there are more than one matched icon in database, only one icon will be
- // returned in the priority of TOUCH_PRECOMPOSED_ICON, TOUCH_ICON, and
- // FAVICON, and the icon type is returned in icon_type parameter if it is not
- // NULL.
+ // and icon type.
// Returns 0 if no entry exists for the specified url.
favicon_base::FaviconID GetFaviconIDForFaviconURL(
const GURL& icon_url,
- int required_icon_type,
- favicon_base::IconType* icon_type);
+ favicon_base::IconType icon_type);
// Gets the icon_url, icon_type and sizes for the specified |icon_id|.
bool GetFaviconHeader(favicon_base::FaviconID icon_id,

Powered by Google App Engine
This is Rietveld 408576698