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

Unified Diff: chrome/browser/ntp_snippets/content_suggestions_service_factory.cc

Issue 2790743003: [Content suggestions] Implement fetching publishers' favicons (Closed)
Patch Set: Minor fix 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
« no previous file with comments | « no previous file | components/ntp_snippets/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
diff --git a/chrome/browser/ntp_snippets/content_suggestions_service_factory.cc b/chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
index 7ef45cc3d093b46b57d76891bee348c77cdf4bc8..012a024bb7e61e82394faffc616290216b300472 100644
--- a/chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
+++ b/chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
@@ -14,6 +14,7 @@
#include "base/time/default_clock.h"
#include "chrome/browser/bookmarks/bookmark_model_factory.h"
#include "chrome/browser/browser_process.h"
+#include "chrome/browser/favicon/large_icon_service_factory.h"
#include "chrome/browser/history/history_service_factory.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/search/suggestions/image_decoder_impl.h"
@@ -272,6 +273,7 @@ ContentSuggestionsServiceFactory::ContentSuggestionsServiceFactory()
BrowserContextDependencyManager::GetInstance()) {
DependsOn(BookmarkModelFactory::GetInstance());
DependsOn(HistoryServiceFactory::GetInstance());
+ DependsOn(LargeIconServiceFactory::GetInstance());
#if defined(OS_ANDROID)
DependsOn(OfflinePageModelFactory::GetInstance());
#endif // OS_ANDROID
@@ -309,12 +311,14 @@ KeyedService* ContentSuggestionsServiceFactory::BuildServiceInstanceFor(
SigninManagerFactory::GetForProfile(profile);
HistoryService* history_service = HistoryServiceFactory::GetForProfile(
profile, ServiceAccessType::EXPLICIT_ACCESS);
+ favicon::LargeIconService* large_icon_service =
+ LargeIconServiceFactory::GetForBrowserContext(profile);
std::unique_ptr<CategoryRanker> category_ranker =
ntp_snippets::BuildSelectedCategoryRanker(
pref_service, base::MakeUnique<base::DefaultClock>());
auto* service = new ContentSuggestionsService(
- State::ENABLED, signin_manager, history_service, pref_service,
- std::move(category_ranker), std::move(user_classifier),
+ State::ENABLED, signin_manager, history_service, large_icon_service,
+ pref_service, std::move(category_ranker), std::move(user_classifier),
std::move(scheduler));
#if defined(OS_ANDROID)
« no previous file with comments | « no previous file | components/ntp_snippets/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698