Remove broken targets from "all" and "gn_all", under Windows component builds.

media/mojo/services/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.
 
 import("//media/media_options.gni")
 import("//services/service_manager/public/cpp/service.gni")
 import("//services/service_manager/public/service_manifest.gni")
 import("//testing/test.gni")
 
 config("mojo_media_config") {
@@ skipping 134 matching lines @@
     ":lib",
     "//mojo/public/c/system",
     "//services/service_manager/public/cpp",
   ]
 
   data_deps = [
     ":media_manifest",
   ]
 }
 
-test("media_service_unittests") {
-  testonly = true
+# crbug.com/676055: media_service_unittests currently fails to link in Windows component builds, so don't declare it, otherwise the "all" target will still try to build it.

[brucedawson, 2016/12/21 18:27:54]

Word-wrap at 80 lines.

[Wez, 2016/12/21 18:54:04]

Done.

+if (!(is_win && is_component_build)) {
+  test("media_service_unittests") {
+    testonly = true
 
-  sources = [
-    "media_service_unittest.cc",
-  ]
+    sources = [
+      "media_service_unittest.cc",
+    ]
 
-  deps = [
-    "//media",
-    "//media/base:test_support",
-    "//media/mojo/clients",
-    "//media/mojo/common",
-    "//media/mojo/interfaces",
-    "//services/service_manager/public/cpp:service_test_support",
-    "//services/service_manager/public/cpp/test:run_all_service_tests",
-    "//testing/gmock",
-    "//testing/gtest",
-  ]
+    deps = [
+      "//media",
+      "//media/base:test_support",
+      "//media/mojo/clients",
+      "//media/mojo/common",
+      "//media/mojo/interfaces",
+      "//services/service_manager/public/cpp:service_test_support",
+      "//services/service_manager/public/cpp/test:run_all_service_tests",
+      "//testing/gmock",
+      "//testing/gtest",
+    ]
 
-  data_deps = [
-    ":media",
-    ":test_manifest",
-  ]
-}
+    data_deps = [
+      ":media",
+      ":test_manifest",
+    ]
+  }
+}  # if (!(is_win && is_component_build))

[watk, 2016/12/21 18:35:36]

total nit, but since you're changing it anyway: this is usually written without
the "if()"

[Wez, 2016/12/21 18:54:04]

That makes sense for preprocessor conditionals, but those end with an explicit
#endif, whereas a closing brace could end a conditional or some other form of
declaration. But OK, done.

 
 service_manifest("media_manifest") {
   name = "media"
   source = "media_manifest.json"
 }
 
 service_manifest("test_manifest") {
   name = "media_service_unittests"
   source = "test_manifest.json"
 }
 
 test("media_pipeline_integration_unittests") {
   testonly = true
 
   deps = [
     "//media/test:mojo_pipeline_integration_tests",
     "//services/service_manager/public/cpp/test:run_all_service_tests",
   ]
 
   data_deps = [
     ":media",
     ":pipeline_test_manifest",
   ]
 }
 
 service_manifest("pipeline_test_manifest") {
   name = "media_pipeline_integration_unittests"
   source = "pipeline_test_manifest.json"
 }