Add ContentSettingsProvider Interface.

chrome/browser/content_settings/content_settings_provider.h

Index: chrome/browser/content_settings/content_settings_provider.h
diff --git a/chrome/browser/content_settings/content_settings_provider.h b/chrome/browser/content_settings/content_settings_provider.h
index e791eaddade8f36ac72730fd3a733fcc4805e030..a832f4c6b0c72162296f368931e510c57e3188ca 100644
--- a/chrome/browser/content_settings/content_settings_provider.h
+++ b/chrome/browser/content_settings/content_settings_provider.h
@@ -8,11 +8,19 @@
 #define CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_PROVIDER_H_
 #pragma once
 
+#include <string>
+#include <vector>
+
+#include "chrome/browser/content_settings/content_settings_pattern.h"
 #include "chrome/common/content_settings.h"
 
-class DefaultContentSettingsProvider {
+class GURL;
+
+namespace content_settings {
+
+class DefaultContentSettingsProviderInterface {
  public:
-  virtual ~DefaultContentSettingsProvider() {}
+  virtual ~DefaultContentSettingsProviderInterface() {}
 
   // True if this provider can provide a default setting for the |content_type|.
   virtual bool CanProvideDefaultSetting(
@@ -39,4 +47,56 @@ class DefaultContentSettingsProvider {
       ContentSettingsType content_type) const = 0;
 };
 
+class ProviderInterface {
+ public:
+  typedef std::string ResourceIdentifier;
+
+  struct Rule {

[Bernhard Bauer, 2011/01/27 11:03:48]

Sorry to create conflicts with jochen's comments, but I think I'd actually
prefer to have Rule outside of ProviderInterface, and even in a separate file.
That way, you can forward-declare it everywhere it's only passed around, and you
also don't need to include content_settings_pattern.h here.

+    Rule() {}
+    Rule(const ContentSettingsPattern& requesting_url_pattern,
+         const ContentSettingsPattern& embedding_url_pattern,
+         ContentSetting& setting)
+    : requesting_url_pattern_(requesting_url_pattern),

[Bernhard Bauer, 2011/01/27 11:03:48]

Nit: Indent two more spaces :)

+      embedding_url_pattern_(embedding_url_pattern),
+      content_setting_(setting) {}
+
+    const ContentSettingsPattern requesting_url_pattern_;
+    const ContentSettingsPattern embedding_url_pattern_;
+    ContentSetting content_setting_;
+  };
+
+  typedef std::vector<Rule> Rules;
+
+  virtual ~ProviderInterface() {}
+
+  virtual bool ContentSettingsTypeIsManaged(
+      ContentSettingsType content_type) = 0;
+
+  // This may be called on any thread.
+  virtual ContentSetting GetContentSetting(
+      const GURL requesting_url,
+      const GURL embedding_url,
+      const ContentSettingsType& content_type,
+      const ResourceIdentifier& resource_identifier) const = 0;
+
+  // This should only be called on the UI thread.
+  virtual void SetContentSetting(
+      const ContentSettingsPattern& requesting_pattern,
+      const ContentSettingsPattern& embedding_pattern,
+      const ContentSettingsType& content_type,
+      const ResourceIdentifier& resource_identifier,
+      ContentSetting content_setting) = 0;
+
+  // This may be called on any thread.
+  virtual void GetAllContentSettingsRules(
+      const ContentSettingsType& content_type,
+      const ResourceIdentifier& resource_identifier,
+      Rules* content_setting_rules) const = 0;
+
+  // This should only be called on the UI thread.
+  virtual void ClearAllContentSettingsRules() = 0;
+};
+
+}  // namespace content_settings
+
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_PROVIDER_H_