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

chrome/browser/content_settings/content_settings_provider_unittest.cc

 // 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.
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/content_settings/content_settings_mock_provider.h"
 #include "chrome/browser/content_settings/content_settings_utils.h"
 #include "googleurl/src/gurl.h"
 
 namespace content_settings {
 
 TEST(ContentSettingsProviderTest, Mock) {

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

Yak shave: This should probably be a ContentSettingsMockProviderTest (and the
file renamed to content_settings_mock_provider_unittest.cc).

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

My new mantra is: Keep the CL as small as possible. I'll do this in a separate
CL. Promised. 

   ContentSettingsPattern pattern =
       ContentSettingsPattern::FromString("[*.]youtube.com");
   GURL url("http://www.youtube.com");
 
   MockProvider mock_provider(
       pattern,
       pattern,
       CONTENT_SETTINGS_TYPE_PLUGINS,
       "java_plugin",
       CONTENT_SETTING_BLOCK,
@@ skipping 21 matching lines @@
                                    CONTENT_SETTINGS_TYPE_PLUGINS,
                                    "flash_plugin", false));
   EXPECT_EQ(CONTENT_SETTING_DEFAULT,
             GetContentSetting(&mock_provider, url, url,
                               CONTENT_SETTINGS_TYPE_GEOLOCATION, "", false));
   EXPECT_EQ(NULL,
             GetContentSettingValue(&mock_provider, url, url,
                                    CONTENT_SETTINGS_TYPE_GEOLOCATION, "",
                                    false));
 
-  mock_provider.SetContentSetting(
+  scoped_ptr<base::Value> value;
+  value.reset(Value::CreateIntegerValue(CONTENT_SETTING_ALLOW));
+  mock_provider.SetWebsiteSetting(
       pattern,
       pattern,
       CONTENT_SETTINGS_TYPE_PLUGINS,
       "java_plugin",
-      CONTENT_SETTING_ALLOW);
+      value.get());
   EXPECT_EQ(CONTENT_SETTING_ALLOW,
             GetContentSetting(&mock_provider, url, url,
                               CONTENT_SETTINGS_TYPE_PLUGINS, "java_plugin",
                               false));
 
+  value.reset(Value::CreateIntegerValue(CONTENT_SETTING_BLOCK));
   mock_provider.set_read_only(true);
-  mock_provider.SetContentSetting(
+  mock_provider.SetWebsiteSetting(
       pattern,
       pattern,
       CONTENT_SETTINGS_TYPE_PLUGINS,
       "java_plugin",
-      CONTENT_SETTING_BLOCK);
+      value.get());
   EXPECT_EQ(CONTENT_SETTING_ALLOW,
             GetContentSetting(&mock_provider, url, url,
                               CONTENT_SETTINGS_TYPE_PLUGINS, "java_plugin",
                               false));
 
   EXPECT_TRUE(mock_provider.read_only());
 
   mock_provider.set_read_only(false);
-  mock_provider.SetContentSetting(
+  mock_provider.SetWebsiteSetting(
       pattern,
       pattern,
       CONTENT_SETTINGS_TYPE_PLUGINS,
       "java_plugin",
-      CONTENT_SETTING_BLOCK);
+      value.get());
   EXPECT_EQ(CONTENT_SETTING_BLOCK,
             GetContentSetting(&mock_provider, url, url,
                               CONTENT_SETTINGS_TYPE_PLUGINS, "java_plugin",
                               false));
 }
 
 }  // namespace content_settings