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

chrome/browser/content_settings/content_settings_utils.cc

Index: chrome/browser/content_settings/content_settings_utils.cc
diff --git a/chrome/browser/content_settings/content_settings_utils.cc b/chrome/browser/content_settings/content_settings_utils.cc
index 2338507437f4729fac112dffde7bd1fb328c1df5..d038d059fced0dab29bbe2c5673c1041131c0fdd 100644
--- a/chrome/browser/content_settings/content_settings_utils.cc
+++ b/chrome/browser/content_settings/content_settings_utils.cc
@@ -121,6 +121,19 @@ bool ParseContentSettingValue(const base::Value* value,
   return *setting != CONTENT_SETTING_DEFAULT;
 }
 
+bool ValueEqualsContentSetting(const base::Value* value,
+                               ContentSetting setting) {

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

Couldn't you just implement this in terms of |ValueToContentSetting|?

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

I removed this method again. Since using ValueToContentSettings solves all
use-cases.

+  if (value) {
+    DCHECK(value->GetType() == base::Value::TYPE_INTEGER);

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

This method DCHECKS if |value| is not an integer FundamentalValue. That's not
what the comment says.

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

Ahhh... these last minute code changes. Anyway this is obsolete now after
removing the method.

+    int int_value;
+    bool success = value->GetAsInteger(&int_value);
+    if (success)
+      return ContentSetting(int_value == setting);

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

Ooooooh. I'm going to pretend I didn't see this line, okay? ;-)

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

No No No I did not write this :(. This was a change in the Matrix. :,(

+    NOTREACHED();
+  }
+  return false;
+}
+
 base::Value* GetContentSettingValueAndPatterns(
     const ProviderInterface* provider,
     const GURL& primary_url,