[inspector] Add swarming support to inspector tests

test/inspector/BUILD.gn

Index: test/inspector/BUILD.gn
diff --git a/test/inspector/BUILD.gn b/test/inspector/BUILD.gn
index cb96bf423ceb972ddbfd6ab829a5b24ec427f879..dcdfe6e8bcb14a5353b8d14c232adad158bf56b2 100644
--- a/test/inspector/BUILD.gn
+++ b/test/inspector/BUILD.gn
@@ -3,37 +3,40 @@
 # found in the LICENSE file.
 
 import("../../gni/v8.gni")
+import("//build_overrides/v8.gni")
 
-v8_executable("inspector-test") {
-  testonly = true
+if (v8_enable_inspector_override) {

[Michael Achenbach, 2016/10/11 18:16:23]

We should maybe just guard the whole gyp target like I did for GN here. Though,
if we have the guard at the deps in all.gyp, it's not really required. Only in
GN, every target will be found on ninja's toplevel all target automatically.

[kozy, 2016/10/11 19:05:57]

Sounds good to me. Could you wrap it in this CL?

+  v8_executable("inspector-test") {
+    testonly = true
 
-  sources = [
-    "inspector-impl.cc",
-    "inspector-impl.h",
-    "inspector-test.cc",
-    "task-runner.cc",
-    "task-runner.h",
-  ]
+    sources = [
+      "inspector-impl.cc",
+      "inspector-impl.h",
+      "inspector-test.cc",
+      "task-runner.cc",
+      "task-runner.h",
+    ]
 
-  configs = [
-    "../..:external_config",
-    "../..:internal_config_base",
-  ]
+    configs = [
+      "../..:external_config",
+      "../..:internal_config_base",
+    ]
 
-  deps = [
-    "../..:v8_libplatform",
-    "//build/config/sanitizers:deps",
-    "//build/win:default_exe_manifest",
-  ]
+    deps = [
+      "../..:v8_libplatform",
+      "//build/config/sanitizers:deps",
+      "//build/win:default_exe_manifest",
+    ]
 
-  if (is_component_build) {
-    # inspector-test can't be built against a shared library, so we
-    # need to depend on the underlying static target in that case.
-    deps += [ "../..:v8_maybe_snapshot" ]
-  } else {
-    deps += [ "../..:v8" ]
-  }
+    if (is_component_build) {
+      # inspector-test can't be built against a shared library, so we
+      # need to depend on the underlying static target in that case.
+      deps += [ "../..:v8_maybe_snapshot" ]
+    } else {
+      deps += [ "../..:v8" ]
+    }
 
-  cflags = []
-  ldflags = []
+    cflags = []
+    ldflags = []
+  }
 }