Origin Identifier Value Map.

chrome/browser/content_settings/content_settings_base_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.
 
 #ifndef CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_BASE_PROVIDER_H_
 #define CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_BASE_PROVIDER_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 23 matching lines @@
 
 // BaseProvider is the abstract base class for all content-settings-provider
 // classes.
 class BaseProvider : public ProviderInterface {
  public:
   // Maps CONTENT_SETTING_ASK for the CONTENT_SETTINGS_TYPE_PLUGINS to
   // CONTENT_SETTING_BLOCK if click-to-play is not enabled.
   static ContentSetting ClickToPlayFixup(ContentSettingsType content_type,
                                          ContentSetting setting);
 
+  // Returns true if the |content_type| requires a resource identifier.
+  static bool RequiresResourceIdentifier(

[Bernhard Bauer, 2011/05/24 14:21:29]

I think that method might be useful to clients of the HostContentSettingsMap as
well, so maybe you could move it there?

[markusheintz_, 2011/05/26 13:22:13]

Strike :) Already committed in a separate CL :)

+      ContentSettingsType content_type);
+
   explicit BaseProvider(bool is_otr);
   virtual ~BaseProvider();
 
 
   // Initializes the Provider.
   virtual void Init() = 0;
 
   virtual ContentSetting GetContentSetting(
       const GURL& requesting_url,
       const GURL& embedding_url,
@@ skipping 11 matching lines @@
       ContentSettingsType content_type,
       const ResourceIdentifier& resource_identifier,
       Rules* content_setting_rules) const;
 
   virtual void ClearAllContentSettingsRules(
       ContentSettingsType content_type) = 0;
 
   virtual void ResetToDefaults() = 0;
 
  protected:
-  // Returns true if the |content_type| requires a resource identifier.
-  bool RequiresResourceIdentifier(
-      ContentSettingsType content_type) const;
-
   // Returns true if the passed |settings| object contains only
   // CONTENT_SETTING_DEFAULT values.
   bool AllDefault(const ExtendedContentSettings& settings) const;
 
   void UpdateContentSettingsMap(
       const ContentSettingsPattern& requesting_pattern,
       const ContentSettingsPattern& embedding_pattern,
       ContentSettingsType content_type,
       const ResourceIdentifier& resource_identifier,
       ContentSetting content_setting);
@@ skipping 32 matching lines @@
   HostContentSettings incognito_settings_;
 
   // Used around accesses to the content_settings_ object to guarantee
   // thread safety.
   mutable base::Lock lock_;
 };
 
 }  // namespace content_settings
 
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_BASE_PROVIDER_H_