Remove content dependencies from content settings providers.

components/content_settings/core/browser/content_settings_provider.h

Index: components/content_settings/core/browser/content_settings_provider.h
diff --git a/components/content_settings/core/browser/content_settings_provider.h b/components/content_settings/core/browser/content_settings_provider.h
index a51ed5246ac231a059940194ed74b710fa4814ec..dc06947c21becf986c6d1dcd877602cd3d747588 100644
--- a/components/content_settings/core/browser/content_settings_provider.h
+++ b/components/content_settings/core/browser/content_settings_provider.h
@@ -11,6 +11,7 @@
 
 #include <string>
 
+#include "base/threading/non_thread_safe.h"
 #include "base/values.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 
@@ -23,7 +24,7 @@ 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() {}