Remove //chrome dependency from ExtensionPrefs

chrome/browser/extensions/api/content_settings/content_settings_service.h

Index: chrome/browser/extensions/api/content_settings/content_settings_service.h
diff --git a/chrome/browser/extensions/api/content_settings/content_settings_service.h b/chrome/browser/extensions/api/content_settings/content_settings_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..91bae43ff8f7eea1c769e19654fec19c3b9864a7
--- /dev/null
+++ b/chrome/browser/extensions/api/content_settings/content_settings_service.h
@@ -0,0 +1,59 @@
+// 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_EXTENSIONS_API_CONTENT_SETTINGS_CONTENT_SETTINGS_SERVICE_H_
+#define CHROME_BROWSER_EXTENSIONS_API_CONTENT_SETTINGS_CONTENT_SETTINGS_SERVICE_H_
+
+#include "base/memory/ref_counted.h"
+#include "extensions/browser/browser_context_keyed_api_factory.h"
+#include "extensions/browser/extension_prefs.h"
+
+namespace extensions {
+
+class ContentSettingsStore;
+
+// This service hosts a single ContentSettingsStore for the
+// chrome.contentSettings API.
+class ContentSettingsService : public BrowserContextKeyedAPI,
+                               public ExtensionPrefs::Observer {
+ public:
+  explicit ContentSettingsService(content::BrowserContext* context);
+  virtual ~ContentSettingsService();
+
+  scoped_refptr<ContentSettingsStore> content_settings_store() const {
+    return content_settings_store_;
+  }
+
+  // Convenience function to get the service for some browser context.
+  static ContentSettingsService* Get(content::BrowserContext* context);
+
+  // BrowserContextKeyedAPI implementation.
+  static BrowserContextKeyedAPIFactory<ContentSettingsService>*
+      GetFactoryInstance();
+
+  // ExtensionPrefs::Observer implementation

[James Cook, 2014/04/01 22:09:37]

nit: either end this one with "." or remove it from the one above

[Ken Rockot(use gerrit already), 2014/04/01 22:41:37]

Done.

+  virtual void OnExtensionRegistered(const std::string& extension_id,
+                                     const base::Time& install_time,
+                                     bool is_enabled) OVERRIDE;
+  virtual void OnExtensionPrefsLoaded(const std::string& extension_id,
+                                      const ExtensionPrefs* prefs) OVERRIDE;
+  virtual void OnExtensionPrefsDeleted(const std::string& extension_id)
+      OVERRIDE;
+  virtual void OnExtensionStateChanged(const std::string& extension_id,
+                                       bool state) OVERRIDE;
+
+ private:
+  friend class BrowserContextKeyedAPIFactory<ContentSettingsService>;
+
+  // BrowserContextKeyedAPI implementation.
+  static const char* service_name() { return "ContentSettingsService"; }

[James Cook, 2014/04/01 22:09:37]

optional style nit: I might just stick this next to the other
BrowserContextKeyedApi stuff above.

[Ken Rockot(use gerrit already), 2014/04/01 22:41:37]

I don't see any fundamental reason for service_name() to stay private, but I'm
going to leave it this way since it's what everyone else does.

+
+  scoped_refptr<ContentSettingsStore> content_settings_store_;
+
+  DISALLOW_COPY_AND_ASSIGN(ContentSettingsService);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_API_CONTENT_SETTINGS_CONTENT_SETTINGS_SERVICE_H_