Replace SetContentSetting method of the content_settings::Provider interface with GetWebsiteSetting

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
 
@@ skipping 25 matching lines @@
   // normal mode. Otherwise, it returns the content settings for the normal
   // mode. The caller takes the ownership of the returned |RuleIterator|. It is
   // not allowed to call other |ProviderInterface| functions (including
   // |GetRuleIterator|) for the same provider until the |RuleIterator| is
   // destroyed.
   virtual RuleIterator* GetRuleIterator(
       ContentSettingsType content_type,
       const ResourceIdentifier& resource_identifier,
       bool incognito) const = 0;
 
-  // Sets the content setting for a particular |primary_pattern|,
-  // |secondary_pattern|, |content_type| tuple. For ContentSettingsTypes that
-  // require a resource identifier to be specified, the |resource_identifier|
-  // must be non-empty.
+  // Askes the provider to set the website setting for a particular
+  // |primary_pattern|, |secondary_pattern|, |content_type| tuple. For
+  // ContentSettingsTypes that require a resource identifier to be specified,
+  // the |resource_identifier| must be non-empty. If the provider accepts the

[Bernhard Bauer, 2011/11/15 15:40:28]

The part about requiring a resource identifier is outdated.

[markusheintz_, 2011/11/15 17:36:58]

Done.

+  // setting it returns true and takes the ownership of the |value|. Otherwise
+  // false is returned and the ownership of the |value| stays with the caller.
   //
   // This should only be called on the UI thread, and not after
   // ShutdownOnUIThread has been called.
-  virtual void SetContentSetting(
+  virtual bool SetWebsiteSetting(
       const ContentSettingsPattern& primary_pattern,
       const ContentSettingsPattern& secondary_pattern,
       ContentSettingsType content_type,
       const ResourceIdentifier& resource_identifier,
-      ContentSetting content_setting) = 0;
+      Value* value) = 0;
 
   // Resets all content settings for the given |content_type| and empty resource
   // identifier to CONTENT_SETTING_DEFAULT.
   //
   // This should only be called on the UI thread, and not after
   // ShutdownOnUIThread has been called.
   virtual void ClearAllContentSettingsRules(
       ContentSettingsType content_type) = 0;
 
   // Detaches the Provider from all Profile-related objects like PrefService.
   // This methods needs to be called before destroying the Profile.
   // Afterwards, none of the methods above that should only be called on the UI
   // thread should be called anymore.
   virtual void ShutdownOnUIThread() = 0;
 };
 
 }  // namespace content_settings
 
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_PROVIDER_H_