Remove MostVisitedSites => SupervisedUserService dep.

chrome/browser/android/ntp/most_visited_sites_bridge.h

Index: chrome/browser/android/ntp/most_visited_sites_bridge.h
diff --git a/chrome/browser/android/ntp/most_visited_sites_bridge.h b/chrome/browser/android/ntp/most_visited_sites_bridge.h
index 9499a1671dc5fa1777a5657713cc88876c6c5d21..804dbac1f125e6af5adc066edf08376c21bdcd68 100644
--- a/chrome/browser/android/ntp/most_visited_sites_bridge.h
+++ b/chrome/browser/android/ntp/most_visited_sites_bridge.h
@@ -15,6 +15,8 @@
 #include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
 #include "chrome/browser/android/ntp/most_visited_sites.h"
+#include "chrome/browser/supervised_user/supervised_user_service.h"
+#include "chrome/browser/supervised_user/supervised_user_service_observer.h"
 
 class Profile;
 
@@ -60,6 +62,27 @@ class MostVisitedSitesBridge {
   class Observer;
   std::unique_ptr<Observer> observer_;

[Marc Treib, 2016/05/09 08:22:48]

Hm, can we rename this to something more explicit, or at least add a comment?
When first reading this, I got confused about all the observers around here.

[sfiera, 2016/05/09 09:33:35]

Done.

 
+  class SupervisorBridge : public MostVisitedSitesSupervisor,
+                           public SupervisedUserServiceObserver {
+   public:
+    explicit SupervisorBridge(Profile* profile);
+
+    void SetObserver(Observer* observer) override;
+    bool IsBlocked(const GURL& url) override;
+    std::vector<MostVisitedSitesSupervisor::Whitelist> whitelists() override;
+    bool IsChildProfile() override;
+
+    // SupervisedUserServiceObserver implementation.
+    void OnURLFilterChanged() override;
+
+   private:
+    Profile* const profile_;
+    Observer* observer_;
+    ScopedObserver<SupervisedUserService, SupervisedUserServiceObserver>
+        register_observer_;
+  };
+  SupervisorBridge supervisor_;
+
   MostVisitedSites most_visited_;
 
   DISALLOW_COPY_AND_ASSIGN(MostVisitedSitesBridge);