Clean content_settings_pattern_parser.* from unnecessary dependencies.

components/content_settings/core/common/content_settings_client.h

Index: components/content_settings/core/common/content_settings_client.h
diff --git a/components/content_settings/core/common/content_settings_client.h b/components/content_settings/core/common/content_settings_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..a8f4bc8f0fffdbdb62e4475ff83349d7ee45319d
--- /dev/null
+++ b/components/content_settings/core/common/content_settings_client.h
@@ -0,0 +1,26 @@
+// Copyright 2014 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 COMPONENTS_CONTENT_SETTINGS_CORE_COMMON_CONTENT_SETTINGS_CLIENT_H_
+#define COMPONENTS_CONTENT_SETTINGS_CORE_COMMON_CONTENT_SETTINGS_CLIENT_H_
+
+#include <string>
+
+#include "base/macros.h"
+
+namespace content_settings {
+
+class ContentSettingsClient {
+ public:
+  ContentSettingsClient() {}
+  virtual ~ContentSettingsClient() {}
+
+  virtual bool IsExtensionScheme(const std::string& scheme);

[vabr (Chromium), 2014/08/06 15:14:01]

The way this method is used, I would rather call it DoesSchemeUsePorts, and
modify it to return true for both extensions and file schemes.

If the notion of the extension scheme is something we want to have in a method
name in the component, then we probably might as well hard-code the value in
here.

[vasilii, 2014/08/11 13:57:18]

url::kFileScheme is used separately in the parser.

+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ContentSettingsClient);
+};
+
+}  // namespace content_settings
+#endif  // COMPONENTS_CONTENT_SETTINGS_CORE_COMMON_CONTENT_SETTINGS_CLIENT_H_