Merge 106270 - Check for default content settings when requiring user authorization for plug-ins.

chrome/browser/content_settings/content_settings_utils.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 "chrome/browser/content_settings/content_settings_utils.h"
 
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 102 matching lines @@
     *setting = CONTENT_SETTING_DEFAULT;
     return true;
   }
   int int_value = -1;
   if (!value->GetAsInteger(&int_value))
     return false;
   *setting = IntToContentSetting(int_value);
   return *setting != CONTENT_SETTING_DEFAULT;
 }
 
+base::Value* GetContentSettingValueAndPatterns(
+    const ProviderInterface* provider,
+    const GURL& primary_url,
+    const GURL& secondary_url,
+    ContentSettingsType content_type,
+    const std::string& resource_identifier,
+    bool include_incognito,
+    ContentSettingsPattern* primary_pattern,
+    ContentSettingsPattern* secondary_pattern) {
+  if (include_incognito) {
+    // Check incognito-only specific settings. It's essential that the
+    // |RuleIterator| gets out of scope before we get a rule iterator for the
+    // normal mode.
+    scoped_ptr<RuleIterator> incognito_rule_iterator(
+        provider->GetRuleIterator(content_type, resource_identifier, true));
+    base::Value* value = GetContentSettingValueAndPatterns(
+        incognito_rule_iterator.get(), primary_url, secondary_url,
+        primary_pattern, secondary_pattern);
+    if (value)
+      return value;
+  }
+  // No settings from the incognito; use the normal mode.
+  scoped_ptr<RuleIterator> rule_iterator(
+      provider->GetRuleIterator(content_type, resource_identifier, false));
+  return GetContentSettingValueAndPatterns(
+      rule_iterator.get(), primary_url, secondary_url,
+      primary_pattern, secondary_pattern);
+}
+
+base::Value* GetContentSettingValueAndPatterns(
+    RuleIterator* rule_iterator,
+    const GURL& primary_url,
+    const GURL& secondary_url,
+    ContentSettingsPattern* primary_pattern,
+    ContentSettingsPattern* secondary_pattern) {
+  while (rule_iterator->HasNext()) {
+    const Rule& rule = rule_iterator->Next();
+    if (rule.primary_pattern.Matches(primary_url) &&
+        rule.secondary_pattern.Matches(secondary_url)) {
+      if (primary_pattern)
+        *primary_pattern = rule.primary_pattern;
+      if (secondary_pattern)
+        *secondary_pattern = rule.secondary_pattern;
+      return rule.value.get()->DeepCopy();
+    }
+  }
+  return NULL;
+}
+
 base::Value* GetContentSettingValue(const ProviderInterface* provider,
                                     const GURL& primary_url,
                                     const GURL& secondary_url,
                                     ContentSettingsType content_type,
                                     const std::string& resource_identifier,
                                     bool include_incognito) {
-  if (include_incognito) {
-    // Check incognito-only specific settings. It's essential that the
-    // |RuleIterator| gets out of scope before we get a rule iterator for the
-    // normal mode.
-    scoped_ptr<RuleIterator> incognito_rule_iterator(
-        provider->GetRuleIterator(content_type, resource_identifier, true));
-    base::Value* value = GetContentSettingValue(
-        incognito_rule_iterator.get(), primary_url, secondary_url);
-    if (value)
-      return value;
-  }
-  // No settings from the incognito; use the normal mode.
-  scoped_ptr<RuleIterator> rule_iterator(
-      provider->GetRuleIterator(content_type, resource_identifier, false));
-  return GetContentSettingValue(
-      rule_iterator.get(), primary_url, secondary_url);
+  return GetContentSettingValueAndPatterns(provider, primary_url, secondary_url,
+                               content_type, resource_identifier,
+                               include_incognito, NULL, NULL);
 }
 
 ContentSetting GetContentSetting(const ProviderInterface* provider,
                                  const GURL& primary_url,
                                  const GURL& secondary_url,
                                  ContentSettingsType content_type,
                                  const std::string& resource_identifier,
                                  bool include_incognito) {
-  if (include_incognito) {
-    // Check incognito-only specific settings. It's essential that the
-    // |RuleIterator| gets out of scope before we get a rule iterator for the
-    // normal mode.
-    scoped_ptr<RuleIterator> incognito_rule_iterator(
-        provider->GetRuleIterator(content_type, resource_identifier, true));
-    ContentSetting setting = GetContentSetting(
-        incognito_rule_iterator.get(), primary_url, secondary_url);
-    if (setting != CONTENT_SETTING_DEFAULT)
-      return setting;
-  }
-  // No settings from the incognito; use the normal mode.
-  scoped_ptr<RuleIterator> rule_iterator(
-      provider->GetRuleIterator(content_type, resource_identifier, false));
-  return GetContentSetting(rule_iterator.get(), primary_url, secondary_url);
-}
-
-base::Value* GetContentSettingValue(RuleIterator* rule_iterator,
-                                    const GURL& primary_url,
-                                    const GURL& secondary_url) {
-  while (rule_iterator->HasNext()) {
-    const Rule& rule = rule_iterator->Next();
-    if (rule.primary_pattern.Matches(primary_url) &&
-        rule.secondary_pattern.Matches(secondary_url)) {
-      return rule.value.get()->DeepCopy();
-    }
-  }
-  return NULL;
-}
-
-ContentSetting GetContentSetting(RuleIterator* rule_iterator,
-                                 const GURL& primary_url,
-                                 const GURL& secondary_url) {
-  while (rule_iterator->HasNext()) {
-    const Rule& rule = rule_iterator->Next();
-    if (rule.primary_pattern.Matches(primary_url) &&
-        rule.secondary_pattern.Matches(secondary_url)) {
-      return ValueToContentSetting(rule.value.get());
-    }
-  }
-  return CONTENT_SETTING_DEFAULT;
+  scoped_ptr<base::Value> value(
+      GetContentSettingValue(provider, primary_url, secondary_url,
+                             content_type, resource_identifier,
+                             include_incognito));
+  return ValueToContentSetting(value.get());
 }
 
 }  // namespace content_settings