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

Unified Diff: components/ntp_tiles/most_visited_sites.cc

Issue 2722133002: [Remote suggestions] Use Google server for fetching MostLikely icons (Closed)
Patch Set: Created 3 years, 10 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/ntp_tiles/most_visited_sites.cc
diff --git a/components/ntp_tiles/most_visited_sites.cc b/components/ntp_tiles/most_visited_sites.cc
index f3695c9946ec90bfe44a7e89456e6e539e820fd9..01f8de306f7210f9a3f81ae3b6b731c8e83d972b 100644
--- a/components/ntp_tiles/most_visited_sites.cc
+++ b/components/ntp_tiles/most_visited_sites.cc
@@ -269,6 +269,9 @@ void MostVisitedSites::BuildCurrentTilesGivenSuggestionsProfile(
tile.whitelist_icon_path = GetWhitelistLargeIconPath(url);
tile.thumbnail_url = GURL(suggestion_pb.thumbnail());
tile.favicon_url = GURL(suggestion_pb.favicon_url());
+ icon_cacher_->StartFetchMostLikely(
+ url, base::Bind(&MostVisitedSites::OnIconMadeAvailable,
+ base::Unretained(this), url));
tiles.push_back(std::move(tile));
}
@@ -359,7 +362,7 @@ NTPTilesVector MostVisitedSites::CreatePopularSitesTiles(
tile.source = NTPTileSource::POPULAR;
popular_sites_tiles.push_back(std::move(tile));
- icon_cacher_->StartFetch(
+ icon_cacher_->StartFetchPopularSites(
popular_site, base::Bind(&MostVisitedSites::OnIconMadeAvailable,
base::Unretained(this), popular_site.url));
if (popular_sites_tiles.size() >= num_popular_sites_tiles)
« no previous file with comments | « components/ntp_tiles/icon_cacher_impl.cc ('k') | ios/chrome/browser/ntp_tiles/ios_most_visited_sites_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698