Add ContentSettingsProvider Interface.

chrome/browser/content_settings/content_settings_provider.h

 // Copyright (c) 2011 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.
 
 // Interface for objects providing content setting rules.
 
 #ifndef CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_PROVIDER_H_
 #define CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_PROVIDER_H_
 #pragma once
 
+#include "chrome/browser/content_settings/content_identifier.h"
+#include "chrome/browser/content_settings/content_settings_rule.h"
+#include "chrome/browser/content_settings/host.h"
+#include "chrome/browser/content_settings/host_pattern.h"

[Bernhard Bauer, 2011/01/26 10:25:18]

If these are classes, I guess you could just forward-declare them.

[markusheintz_, 2011/01/26 15:27:08]

Will fix when clear which of this classes stay.

 #include "chrome/common/content_settings.h"
 
+using content_settings::ContentIdentifier;

[Bernhard Bauer, 2011/01/26 10:25:18]

We don't use namespace imports in header files, only in implementation files, in
order not to pollute the namespaces of every files that includes this header.

You could put these classes into the content_settings namespace though.

[markusheintz_, 2011/01/26 15:27:08]

I put the ContentSettingsProvider in the content_settings namespace.

+using content_settings::ContentSettingsRule;
+using content_settings::ContentSettingsRulePtr;
+using content_settings::ContentSettingsRules;
+using content_settings::Host;
+using content_settings::HostPattern;
+
+class GURL;
+
 class DefaultContentSettingsProvider {
  public:
   virtual ~DefaultContentSettingsProvider() {}
 
   // True if this provider can provide a default setting for the |content_type|.
   virtual bool CanProvideDefaultSetting(
       ContentSettingsType content_type) const = 0;
 
   // Returns the default content setting this provider has for the given
   // |content_type|, or CONTENT_SETTING_DEFAULT if nothing be provided for this
   // type.
   virtual ContentSetting ProvideDefaultSetting(
       ContentSettingsType content_type) const = 0;
 
   // Notifies the provider that the host content settings map would like to
   // update the default setting for the given |content_type|. The provider may
   // ignore this.
   virtual void UpdateDefaultSetting(ContentSettingsType content_type,
                                     ContentSetting setting) = 0;
 
   // Resets the state of the provider to the default.
   virtual void ResetToDefaults() = 0;
 
   // True if the default setting for the |content_type| is policy managed, i.e.,
   // there shouldn't be any UI shown to modify this setting.
   virtual bool DefaultSettingIsManaged(
       ContentSettingsType content_type) const = 0;
 };
 
+class ContentSettingsProvider {

[jochen (gone - plz use gerrit), 2011/01/26 11:29:18]

public: missing

[markusheintz_, 2011/01/26 15:27:08]

Done.

+  virtual ~ContentSettingsProvider() {}

[jochen (gone - plz use gerrit), 2011/01/26 11:29:18]

where is isManaged, isReadOnly, and canProvide?

Probably also getProviderType and enum { USER_PREFS, POLICIES, WEBAPPS }

[Bernhard Bauer, 2011/01/26 11:34:23]

We don't need it anymore with the new interface :) For example, the only
writable ContentSettingProvider is the one reading from user preferences, so the
HostContentSettingsMap can just keep track of it separately. 

[markusheintz_, 2011/01/26 15:27:08]

Don't need them.

I think the enum should live in the HCSM. I guess it's enougth. Don't think the
provider needs such a method as long as the HCSP knows which provider is which
that is fine.

+
+  // Returns the ContentSetting for the given |host| and |content_identifier|.
+  //

[Bernhard Bauer, 2011/01/26 10:25:18]

Nit: remove empty comment lines.

[markusheintz_, 2011/01/26 15:27:08]

Done.

+  // This may be called on any thread.
+  virtual ContentSetting GetContentSetting(
+      const Host& host,
+      const ContentIdentifier& content_identifier) const = 0;
+
+  // Sets the |content_setting| for the given |host| and |content_identifier|.
+  // Setting the value to CONTENT_SETTING_DEFAULT causes the default setting for
+  // that type to be used when loading pages matching this pattern.
+  //
+  // This should only be called on the UI thread.
+  virtual void SetContentSetting(
+      const Host& host,

[Bernhard Bauer, 2011/01/26 10:25:18]

Should this take a pattern?

[markusheintz_, 2011/01/26 15:27:08]

 of course! :)

done

+      const ContentIdentifier& content_identifier,
+      ContentSetting content_setting) = 0;
+
+  // For the given |content_identifier| return all content-setting-rules with a
+  // non-default setting.
+  //
+  // This may be called on any thread.
+  virtual void GetAllContentSettingsRules(
+      const ContentIdentifier& content_identifier,
+      ContentSettingsRules* content_setting_rules) const = 0;
+
+  // Clears all content-setting-rules for the given |content_identifier|.
+  //
+  // This should only be called on the UI thread.
+  virtual void ClearAllContentSettingsRules(
+      const ContentIdentifier& content_identifier) = 0;
+};
+
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_PROVIDER_H_