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

Unified Diff: chrome/browser/content_settings/content_settings_pref_provider.h

Issue 7344008: Make the hcsm and its providers communicate via an observer interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: " Created 9 years, 5 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/content_settings/content_settings_pref_provider.h
diff --git a/chrome/browser/content_settings/content_settings_pref_provider.h b/chrome/browser/content_settings/content_settings_pref_provider.h
index c30e4049b364308c1da58b3b99006eaa1d4056a2..6c51ecbb4703c5442c1f1e57a2d172a932f53e21 100644
--- a/chrome/browser/content_settings/content_settings_pref_provider.h
+++ b/chrome/browser/content_settings/content_settings_pref_provider.h
@@ -15,7 +15,7 @@
#include "base/basictypes.h"
#include "base/synchronization/lock.h"
#include "chrome/browser/content_settings/content_settings_origin_identifier_value_map.h"
-#include "chrome/browser/content_settings/content_settings_provider.h"
+#include "chrome/browser/content_settings/content_settings_observable_provider.h"
#include "chrome/browser/content_settings/content_settings_utils.h"
#include "chrome/browser/prefs/pref_change_registrar.h"
#include "content/common/notification_observer.h"
@@ -99,13 +99,12 @@ class PrefDefaultProvider : public DefaultProviderInterface,
// Content settings provider that provides content settings from the user
// preference.
-class PrefProvider : public ProviderInterface,
+class PrefProvider : public ObservableProvider,
public NotificationObserver {
public:
static void RegisterUserPrefs(PrefService* prefs);
- PrefProvider(HostContentSettingsMap* map,
- PrefService* prefs,
+ PrefProvider(PrefService* prefs,
bool incognito);
virtual ~PrefProvider();
@@ -189,14 +188,9 @@ class PrefProvider : public ProviderInterface,
static void CanonicalizeContentSettingsExceptions(
DictionaryValue* all_settings_dictionary);
- void NotifyObservers(const ContentSettingsDetails& details);
-
// Weak; owned by the Profile and reset in ShutdownOnUIThread.
PrefService* prefs_;
- // Weak; owns us
- HostContentSettingsMap* host_content_settings_map_;
-
bool is_incognito_;
PrefChangeRegistrar pref_change_registrar_;

Powered by Google App Engine
This is Rietveld 408576698