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

Unified Diff: components/favicon/core/favicon_handler.h

Issue 2948963002: Prefer 192x192 icons from Web Manifests instead of 144x144 (Closed)
Patch Set: Created 3 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: components/favicon/core/favicon_handler.h
diff --git a/components/favicon/core/favicon_handler.h b/components/favicon/core/favicon_handler.h
index 5b4c7aa3bab56003121b3c2717c0e2cceaafbb82..610ffbcf5690086f2a471965a186c8e3acc63c21 100644
--- a/components/favicon/core/favicon_handler.h
+++ b/components/favicon/core/favicon_handler.h
@@ -206,7 +206,8 @@ class FaviconHandler {
// Called when the actual list of favicon candidates to be processed is
// available, which can be either icon URLs listed in the HTML head instead
// or, if a Web Manifest was provided, the list of icons there.
- void OnGotFinalIconURLCandidates(const std::vector<FaviconURL>& candidates);
+ void OnGotFinalIconURLCandidates(const std::vector<FaviconURL>& candidates,
+ const std::vector<int>& desired_pixel_sizes);
// Called when the history request for favicon data mapped to |url_| has
// completed and the renderer has told us the icon URLs used by |url_|
« no previous file with comments | « no previous file | components/favicon/core/favicon_handler.cc » ('j') | components/favicon/core/favicon_handler_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698