Add a HostContentSettingsMap layer for Supervised Users.

chrome/browser/supervised_user/supervised_user_settings_service.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SETTINGS_SERVICE_H_
 #define CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SETTINGS_SERVICE_H_
 
 #include <string>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/prefs/pref_store.h"
 #include "base/values.h"
 #include "chrome/browser/supervised_user/supervised_users.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "sync/api/syncable_service.h"
 
+class HostContentSettingsMap;
 class PersistentPrefStore;
 class Profile;
 
 namespace base {
 class FilePath;
 class SequencedTaskRunner;
 }
 
 // This class syncs supervised user settings from a server, which are mapped to
 // preferences. The downloaded settings are persisted in a PrefStore (which is
@@ skipping 41 matching lines @@
 
   // Initializes the service by loading its settings from the |pref_store|.
   // Use this method in tests to inject a different PrefStore than the
   // default one.
   void Init(scoped_refptr<PersistentPrefStore> pref_store);
 
   // Adds a callback to be called when supervised user settings are initially
   // available, or when they change.
   void Subscribe(const SettingsCallback& callback);
 
+  // Register own content settings provider with the specified map.
+  void RegisterContentSettings(

[Bernhard Bauer, 2015/02/26 12:44:58]

Why does the SupervisedUserSettingsService now have to know about the
HostContentSettingsMap? The dependencies should be the other way around:

HostContentSettingsMap (or its embedder, i.e. Profile) -> SupervisedProvider ->
SupervisedUserSettingsService

[knn, 2015/02/26 16:18:56]

HostContentSettingsMap is a component so it cannot depend on the
SupervisedProvider. The SupervisedUserSettingsService seemed like a nice
facilitator for this dependency instead of doing it on a profile.
I just uploaded the cl to check, I'll revert back to using the profile then.

[Bernhard Bauer, 2015/02/26 18:03:54]

Hm, I'm wondering whether we could have an embedder method to insert the
individual providers into the HCSM. I'll talk to vabr@ (who I think owns HCSM
componentization about it).

+      HostContentSettingsMap* host_content_settings_map);
+
   // Activates/deactivates the service. This is called by the
   // SupervisedUserService when it is (de)activated.
   void SetActive(bool active);
 
   // Whether supervised user settings are available.
   bool IsReady();
 
   // Clears all supervised user settings and items.
   void Clear();
 
@@ skipping 66 matching lines @@
 
   std::vector<SettingsCallback> subscribers_;
 
   scoped_ptr<syncer::SyncChangeProcessor> sync_processor_;
   scoped_ptr<syncer::SyncErrorFactory> error_handler_;
 
   DISALLOW_COPY_AND_ASSIGN(SupervisedUserSettingsService);
 };
 
 #endif  // CHROME_BROWSER_SUPERVISED_USER_SUPERVISED_USER_SETTINGS_SERVICE_H_