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|,
+  // Sets 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.
+  // must be non-empty. The ownership of the passed |value| remains with the
+  // caller. Provider copy the passed |value|.

[Bernhard Bauer, 2011/11/11 14:00:40]

The last sentence is unnecessary (what the provider does with the passed |value|
is an implementation detail).

[markusheintz_, 2011/11/14 11:15:10]

Done.

   //
   // This should only be called on the UI thread, and not after
   // ShutdownOnUIThread has been called.
-  virtual void SetContentSetting(
+  virtual void SetWebsiteSetting(
       const ContentSettingsPattern& primary_pattern,
       const ContentSettingsPattern& secondary_pattern,
       ContentSettingsType content_type,
       const ResourceIdentifier& resource_identifier,
-      ContentSetting content_setting) = 0;
+      const 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_