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

Unified Diff: chrome/browser/ui/app_list/search/suggestions/url_suggestion_result.cc

Issue 2796343003: Remove preferences among favicon types when choosing large icons
Patch Set: Rebased. 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: chrome/browser/ui/app_list/search/suggestions/url_suggestion_result.cc
diff --git a/chrome/browser/ui/app_list/search/suggestions/url_suggestion_result.cc b/chrome/browser/ui/app_list/search/suggestions/url_suggestion_result.cc
index a076845bfe29b2c019630adab86fe405a66b566a..ccbe9e9f421333774fe2929aff1edce93ff93054 100644
--- a/chrome/browser/ui/app_list/search/suggestions/url_suggestion_result.cc
+++ b/chrome/browser/ui/app_list/search/suggestions/url_suggestion_result.cc
@@ -59,15 +59,10 @@ std::unique_ptr<SearchResult> URLSuggestionResult::Duplicate() const {
}
void URLSuggestionResult::UpdateIcon() {
- std::vector<int> icon_types;
- icon_types.push_back(favicon_base::IconType::FAVICON);
- icon_types.push_back(favicon_base::IconType::TOUCH_ICON);
- icon_types.push_back(favicon_base::IconType::TOUCH_PRECOMPOSED_ICON);
-
if (favicon_service_) {
// NOTE: Favicons with size < kMinimumDesiredSizePixels are still returned.
favicon_service_->GetLargestRawFaviconForPageURL(
- GURL(suggestion_.url()), icon_types, kTileIconSize,
+ GURL(suggestion_.url()), kTileIconSize,
base::Bind(&URLSuggestionResult::OnDidGetIcon, base::Unretained(this)),
cancelable_task_tracker_.get());
}
« no previous file with comments | « chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc ('k') | components/favicon/core/favicon_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698