Clean content_settings_pattern_parser.* from unnecessary dependencies.

components/content_settings/core/common/embedder_variables.cc

+// Copyright (c) 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.
+
+#include "components/content_settings/core/common/embedder_variables.h"
+
+#include "base/logging.h"
+
+namespace content_settings {
+
+const char* extension_scheme = NULL;
+
+void SetNonPortScheme(const char* scheme) {
+  extension_scheme = scheme;
+}
+
+// Compares |scheme| against embedder's extension scheme.
+bool IsNonPortScheme(const std::string& scheme) {
+  DCHECK(extension_scheme);

[vabr (Chromium), 2014/08/20 09:52:22]

optional nit: This DCHECK enforces that SetNonPortScheme is called before, but
the comment for SetNonPortScheme only says "should be called". I suggest
phrasing that comment as "Needs to be called with a non-NULL pointer" or
similarly.

[vasilii, 2014/08/21 14:58:02]

Done.

+  return scheme == extension_scheme;
+}
+
+}  // namespace content_settings