Pull g_browser_process out of PopularSites.

chrome/browser/android/ntp/most_visited_sites.h

Index: chrome/browser/android/ntp/most_visited_sites.h
diff --git a/chrome/browser/android/ntp/most_visited_sites.h b/chrome/browser/android/ntp/most_visited_sites.h
index 467983b9438e5492d71d722b658d387c852b80b0..7749df3bc1a22d6c9b87e234c286bde85231d40c 100644
--- a/chrome/browser/android/ntp/most_visited_sites.h
+++ b/chrome/browser/android/ntp/most_visited_sites.h
@@ -33,8 +33,9 @@ namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
-class PopularSites;
-class Profile;

[Marc Treib, 2016/04/29 14:46:34]

Profile one is actually still required for now.

[sfiera, 2016/04/29 14:50:54]

Oh yeah, I guess we must be picking it up from SUS.

In any case, CL is already out to you that will actually get rid of Profile from
this file.

+namespace variations {
+class VariationsService;
+}
 
 // Tracks the list of most visited sites and their thumbnails.
 //
@@ -85,7 +86,8 @@ class MostVisitedSites : public history::TopSitesObserver,
     DISALLOW_COPY_AND_ASSIGN(Suggestion);
   };
 
-  explicit MostVisitedSites(Profile* profile);
+  explicit MostVisitedSites(Profile* profile,
+                            variations::VariationsService* variations_service);

[Bernhard Bauer, 2016/04/29 14:35:45]

Oops, forgot: This doesn't need to be explicit anymore now.

[sfiera, 2016/04/29 14:50:54]

Done.

 
   ~MostVisitedSites() override;
 
@@ -212,6 +214,7 @@ class MostVisitedSites : public history::TopSitesObserver,
   // The profile whose most visited sites will be queried.
   Profile* profile_;
 
+  variations::VariationsService* variations_service_;
   scoped_refptr<history::TopSites> top_sites_;
   suggestions::SuggestionsService* suggestions_service_;