Fix path_parser dependency on policy component

chrome/browser/policy/BUILD.gn

 # 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.
 
-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",
+    ]
+  }
 }