Remove content dependencies from content settings providers.

components/content_settings/core/browser/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 COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_CONTENT_SETTINGS_PROVIDER_H_
 #define COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_CONTENT_SETTINGS_PROVIDER_H_
 
 #define NO_RESOURCE_IDENTIFIER std::string()
 
 #include <string>
 
+#include "base/threading/non_thread_safe.h"
 #include "base/values.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 
 class ContentSettingsPattern;
 
 namespace content_settings {
 
 struct Rule;
 class RuleIterator;
 
 typedef std::string ResourceIdentifier;
 
-class ProviderInterface {
+class ProviderInterface : public base::NonThreadSafe {

[Bernhard Bauer, 2014/09/23 10:08:19]

This class is supposed to be an empty interface, and deriving from NonThreadSafe
(or adding a ThreadChecker member) would change that.

[Jun Mukai, 2014/09/23 20:48:45]

Removed

  public:
   virtual ~ProviderInterface() {}
 
   // Returns a |RuleIterator| over the content setting rules stored by this
   // provider. If |incognito| is true, the iterator returns only the content
   // settings which are applicable to the incognito mode and differ from the
   // 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
@@ skipping 29 matching lines @@
   // 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  // COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_CONTENT_SETTINGS_PROVIDER_H_