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

Unified Diff: components/ntp_tiles/popular_sites_impl.h

Issue 2695713004: Add baked-in favicons for default popular sites on NTP (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/popular_sites_impl.h
diff --git a/components/ntp_tiles/popular_sites_impl.h b/components/ntp_tiles/popular_sites_impl.h
index 17280da01a417e33756a6ded4d3dc0174f1f3a71..63e5f4d215a92125b93a76d2ea109ef32c8dd5c7 100644
--- a/components/ntp_tiles/popular_sites_impl.h
+++ b/components/ntp_tiles/popular_sites_impl.h
@@ -26,6 +26,10 @@ namespace net {
class URLRequestContextGetter;
}
+namespace favicon {
+class FaviconService;
+}
+
namespace user_prefs {
class PrefRegistrySyncable;
}
@@ -53,6 +57,7 @@ class PopularSitesImpl : public PopularSites, public net::URLFetcherDelegate {
PrefService* prefs,
const TemplateURLService* template_url_service,
variations::VariationsService* variations_service,
+ favicon::FaviconService* favicon_service,
net::URLRequestContextGetter* download_context,
const base::FilePath& directory,
ParseJSONCallback parse_json);

Powered by Google App Engine
This is Rietveld 408576698