Fix path_parser dependency on policy component

chrome/browser/policy/BUILD.gn

Index: chrome/browser/policy/BUILD.gn
diff --git a/chrome/browser/policy/BUILD.gn b/chrome/browser/policy/BUILD.gn
index e3b8e91cd656b260b78a1c56c80ae5cb81f6d33d..14267d3b0f28634acc23db03313a1289055084f0 100644
--- a/chrome/browser/policy/BUILD.gn
+++ b/chrome/browser/policy/BUILD.gn
@@ -2,17 +2,21 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-source_set("path_parser") {
-  sources = [
-    "policy_path_parser.h",
-    "policy_path_parser_linux.cc",
-    "policy_path_parser_mac.mm",
-    "policy_path_parser_win.cc",
-  ]
+import("//build/config/features.gni")
 
-  deps = [
-    "//base",
-    "//chrome/common:constants",
-    "//components/policy",
-  ]
+if (enable_configuration_policy) {

[brettw, 2015/12/01 17:49:06]

This if statement doesn't do anything. The only way GN will load a BUILD file is
if somebody references a target in it, and if configuration policy is disabled,
there will be no targets.

If you're trying to prevent accidents, the best thing to do is add at the top:

assert(enable_configuration_policy)

[Tomasz Moniuszko, 2015/12/02 10:23:05]

Done.

+  source_set("path_parser") {
+    sources = [
+      "policy_path_parser.h",
+      "policy_path_parser_linux.cc",
+      "policy_path_parser_mac.mm",
+      "policy_path_parser_win.cc",
+    ]
+
+    deps = [
+      "//base",
+      "//chrome/common:constants",
+      "//components/policy",
+    ]
+  }
 }