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

Unified Diff: chrome/browser/android/ntp/most_visited_sites.cc

Issue 1930413002: Pull g_browser_process out of PopularSites. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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/android/ntp/most_visited_sites.cc
diff --git a/chrome/browser/android/ntp/most_visited_sites.cc b/chrome/browser/android/ntp/most_visited_sites.cc
index ac625b2a27cddf63298249efe447229b86534716..3c8c78fd3785d07625dbd0a69b08d28fd2137ad2 100644
--- a/chrome/browser/android/ntp/most_visited_sites.cc
+++ b/chrome/browser/android/ntp/most_visited_sites.cc
@@ -177,8 +177,11 @@ MostVisitedSites::Suggestion::Suggestion(Suggestion&&) = default;
MostVisitedSites::Suggestion&
MostVisitedSites::Suggestion::operator=(Suggestion&&) = default;
-MostVisitedSites::MostVisitedSites(Profile* profile)
- : profile_(profile), top_sites_(TopSitesFactory::GetForProfile(profile)),
+MostVisitedSites::MostVisitedSites(
+ Profile* profile,
+ variations::VariationsService* variations_service)
+ : profile_(profile), variations_service_(variations_service),
+ top_sites_(TopSitesFactory::GetForProfile(profile)),
suggestions_service_(SuggestionsServiceFactory::GetForProfile(profile_)),
observer_(nullptr), num_sites_(0),
received_most_visited_sites_(false), received_popular_sites_(false),
@@ -209,6 +212,7 @@ void MostVisitedSites::SetMostVisitedURLsObserver(
popular_sites_.reset(new PopularSites(
profile_->GetPrefs(),
TemplateURLServiceFactory::GetForProfile(profile_),
+ variations_service_,
profile_->GetRequestContext(),
GetPopularSitesCountry(),
GetPopularSitesVersion(),
« no previous file with comments | « chrome/browser/android/ntp/most_visited_sites.h ('k') | chrome/browser/android/ntp/most_visited_sites_bridge.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698