Move gn _run target generation into test(). (RELAND)

This is a reland of https://codereview.chromium.org/1653003006/

Consolidating the _run targets into the test template should make them
easier to maintain (and, eventually, remove).

BUG=583694
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review URL: https://codereview.chromium.org/1662053002

Cr-Commit-Position: refs/heads/master@{#373385}

BUILD.gn

@@ -462,8 +462,8 @@ group("both_gn_and_gyp") {
         # TODO(dpranke): Figure out what platforms should actually have this.
         deps += [
           "//components/nacl/loader:helper_nonsfi",
-          "//components/nacl/loader:helper_nonsfi_unittests",
           "//components/nacl/loader:nacl_helper",
+          "//components/nacl/loader:nacl_helper_nonsfi_unittests",
         ]
       }
     }

base/BUILD.gn

@@ -1522,15 +1522,6 @@ if (is_win) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("base_unittests_run") {
-  testonly = true
-  deps = [
-    ":base_unittests",
-  ]
-}
-
 test("base_unittests") {
   sources = [
     "allocator/tcmalloc_unittest.cc",

blimp/BUILD.gn

@@ -59,12 +59,3 @@ test("blimp_unittests") {
     deps += [ "//blimp/engine:unit_tests" ]
   }
 }
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("blimp_unittests_run") {
-  testonly = true
-  deps = [
-    ":blimp_unittests",
-  ]
-}

cc/BUILD.gn

@@ -756,15 +756,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("cc_unittests_run") {
-  testonly = true
-  deps = [
-    ":cc_unittests",
-  ]
-}
-
 test("cc_unittests") {
   sources = [
     "animation/animation_host_unittest.cc",

chrome/test/BUILD.gn

@@ -285,15 +285,6 @@ if (!is_android) {
     ]
   }
 
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("interactive_ui_tests_run") {
-    testonly = true
-    deps = [
-      ":interactive_ui_tests",
-    ]
-  }
-
   test("interactive_ui_tests") {
     sources =
         rebase_path(chrome_tests_gypi_values.chrome_interactive_ui_test_sources,
@@ -779,15 +770,6 @@ if (!is_android) {
     ]
   }
 
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("browser_tests_run") {
-    testonly = true
-    data_deps = [
-      ":browser_tests",
-    ]
-  }
-
   test("browser_tests") {
     sources = [
       "base/browser_tests_main.cc",
@@ -1275,15 +1257,6 @@ if (!is_android) {
     }
   }
 
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("sync_integration_tests_run") {
-    testonly = true
-    deps = [
-      ":sync_integration_tests",
-    ]
-  }
-
   test("sync_integration_tests") {
     sources =
         rebase_path(chrome_tests_gypi_values.sync_integration_tests_sources,
@@ -1482,15 +1455,6 @@ js2gtest("unit_tests_js") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("unit_tests_run") {
-  testonly = true
-  deps = [
-    ":unit_tests",
-  ]
-}
-
 test("unit_tests") {
   sources = rebase_path(chrome_tests_unit_gypi_values.chrome_unit_tests_sources,
                         ".",

chrome/test/chromedriver/BUILD.gn

@@ -190,15 +190,6 @@ executable("chromedriver") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("chromedriver_unittests_run") {
-  testonly = true
-  deps = [
-    ":chromedriver_unittests",
-  ]
-}
-
 test("chromedriver_unittests") {
   sources =
       rebase_path(chrome_tests_gypi_values.chrome_driver_unittests_sources,

components/BUILD.gn

@@ -12,15 +12,6 @@ if (is_android) {
   import("//build/config/android/rules.gni")
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("components_unittests_run") {
-  testonly = true
-  deps = [
-    ":components_unittests",
-  ]
-}
-
 # To add a unit test to this target, make a "unit_test" source_set in your
 # component (it's important to use a source_set instead of a static library or
 # no tests will run) and add a reference here. You can add more than one unit
@@ -330,15 +321,6 @@ if (is_android) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("components_browsertests_run") {
-  testonly = true
-  deps = [
-    ":components_browsertests",
-  ]
-}
-
 test("components_browsertests") {
   sources = []
   defines = [ "HAS_OUT_OF_PROC_TEST_RUNNER" ]

components/nacl/loader/BUILD.gn

@@ -71,15 +71,6 @@ source_set("loader") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("nacl_loader_unittests_run") {
-  testonly = true
-  deps = [
-    ":nacl_loader_unittests",
-  ]
-}
-
 test("nacl_loader_unittests") {
   sources = [
     "nacl_ipc_adapter_unittest.cc",
@@ -156,6 +147,9 @@ if (is_linux) {
       "//base",
       "//base/test:test_launcher_nacl_nonsfi",
     ]
+    data_deps = [
+      ":nacl_helper_nonsfi_unittests_main_copy(//build/toolchain/nacl:newlib_pnacl_nonsfi)",
+    ]
   }
 
   group("helper_nonsfi") {
@@ -163,24 +157,6 @@ if (is_linux) {
       ":nacl_helper_nonsfi_copy(//build/toolchain/nacl:newlib_pnacl_nonsfi)",
     ]
   }
-
-  group("helper_nonsfi_unittests") {
-    testonly = true
-    data_deps = [
-      ":nacl_helper_nonsfi_unittests",
-      ":nacl_helper_nonsfi_unittests_main_copy(//build/toolchain/nacl:newlib_pnacl_nonsfi)",
-    ]
-  }
-
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("nacl_helper_nonsfi_unittests_run") {
-    testonly = true
-    data_deps = [
-      ":helper_nonsfi_unittests",
-      ":nacl_helper_nonsfi_unittests_main_copy(//build/toolchain/nacl:newlib_pnacl_nonsfi)",
-    ]
-  }
 }
 
 if (is_win && target_cpu == "x86" && current_cpu == "x64") {

content/test/BUILD.gn

@@ -339,15 +339,6 @@ if (is_android) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("content_browsertests_run") {
-  testonly = true
-  deps = [
-    ":content_browsertests",
-  ]
-}
-
 test("content_browsertests") {
   sources = rebase_path(content_tests_gypi_values.content_browsertests_sources,
                         ".",
@@ -560,15 +551,6 @@ test("content_browsertests") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("content_unittests_run") {
-  testonly = true
-  deps = [
-    ":content_unittests",
-  ]
-}
-
 test("content_unittests") {
   defines = []
   sources = rebase_path(content_tests_gypi_values.content_unittests_sources,
@@ -835,15 +817,6 @@ test("content_perftests") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("content_gl_tests_run") {
-  testonly = true
-  deps = [
-    ":content_gl_tests",
-  ]
-}
-
 test("content_gl_tests") {
   sources = [
     "../common/gpu/client/gl_helper_unittest.cc",

crypto/BUILD.gn

@@ -226,15 +226,6 @@ if (false && is_win) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("crypto_unittests_run") {
-  testonly = true
-  deps = [
-    ":crypto_unittests",
-  ]
-}
-
 test("crypto_unittests") {
   sources = [
     "aead_openssl_unittest.cc",

device/BUILD.gn

@@ -16,15 +16,6 @@ if (is_mac) {
 is_linux_without_udev = is_linux && !use_udev
 is_linux_without_dbus = is_linux && !use_dbus
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("device_unittests_run") {
-  testonly = true
-  deps = [
-    ":device_unittests",
-  ]
-}
-
 test("device_unittests") {
   sources = [
     "battery/battery_status_manager_win_unittest.cc",

extensions/BUILD.gn

@@ -141,15 +141,6 @@ repack("shell_and_test_pak") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("extensions_unittests_run") {
-  testonly = true
-  deps = [
-    ":extensions_unittests",
-  ]
-}
-
 test("extensions_unittests") {
   sources =
       rebase_path(extensions_tests_gypi_values.extensions_unittests_sources,
@@ -218,15 +209,6 @@ test("extensions_unittests") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("extensions_browsertests_run") {
-  testonly = true
-  deps = [
-    ":extensions_browsertests",
-  ]
-}
-
 test("extensions_browsertests") {
   sources =
       rebase_path(extensions_tests_gypi_values.extensions_browsertests_sources,

extensions/shell/BUILD.gn

@@ -145,15 +145,6 @@ if (!(is_chromeos && !use_ozone)) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("app_shell_unittests_run") {
-  testonly = true
-  deps = [
-    ":app_shell_unittests",
-  ]
-}
-
 test("app_shell_unittests") {
   sources = rebase_path(app_shell_gypi_values.app_shell_unittests_sources,
                         ".",

gin/BUILD.gn

@@ -202,12 +202,3 @@ test("gin_unittests") {
     ":gin_shell",
   ]
 }
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("gin_unittests_run") {
-  testonly = true
-  deps = [
-    ":gin_unittests",
-  ]
-}

google_apis/BUILD.gn

@@ -224,15 +224,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("google_apis_unittests_run") {
-  testonly = true
-  deps = [
-    ":google_apis_unittests",
-  ]
-}
-
 test("google_apis_unittests") {
   sources = [
     "gaia/gaia_auth_fetcher_unittest.cc",

google_apis/gcm/BUILD.gn

@@ -123,15 +123,6 @@ executable("mcs_probe") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("gcm_unit_tests_run") {
-  testonly = true
-  deps = [
-    ":gcm_unit_tests",
-  ]
-}
-
 test("gcm_unit_tests") {
   sources = [
     "base/mcs_message_unittest.cc",

gpu/BUILD.gn

@@ -101,13 +101,6 @@ source_set("test_support") {
   ]
 }
 
-group("gl_tests_run") {
-  testonly = true
-  deps = [
-    ":gl_tests",
-  ]
-}
-
 test("gl_tests") {
   sources = [
     "command_buffer/tests/compressed_texture_test.cc",
@@ -177,15 +170,6 @@ test("gl_tests") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("gpu_unittests_run") {
-  testonly = true
-  deps = [
-    ":gpu_unittests",
-  ]
-}
-
 test("gpu_unittests") {
   sources = [
     "command_buffer/client/buffer_tracker_unittest.cc",
@@ -328,23 +312,3 @@ test("gpu_perftests") {
     "//ui/gl",
   ]
 }
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("angle_unittests_run") {
-  testonly = true
-  deps = [
-    "//third_party/angle/src/tests:angle_unittests",
-  ]
-}
-
-if (is_win || is_linux || is_mac) {
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("angle_end2end_tests_run") {
-    testonly = true
-    deps = [
-      "//third_party/angle/src/tests:angle_end2end_tests",
-    ]
-  }
-}

gpu/gles2_conform_support/BUILD.gn

@@ -174,15 +174,6 @@ if (internal_gles2_conform_tests) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("gles2_conform_test_run") {
-  testonly = true
-  deps = [
-    ":gles2_conform_test",
-  ]
-}
-
 test("gles2_conform_test") {
   sources = [
     "gles2_conform_test.cc",

ipc/BUILD.gn

@@ -144,13 +144,6 @@ source_set("param_traits") {
   ]
 }
 
-group("ipc_tests_run") {
-  testonly = true
-  deps = [
-    ":ipc_tests",
-  ]
-}
-
 test("ipc_tests") {
   sources = [
     "attachment_broker_mac_unittest.cc",

jingle/BUILD.gn

@@ -121,15 +121,6 @@ if (enable_webrtc || !is_android) {
     ]
   }
 
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("jingle_unittests_run") {
-    testonly = true
-    deps = [
-      ":jingle_unittests",
-    ]
-  }
-
   # GYP version: jingle/jingle.gyp:jingle_unittests
   test("jingle_unittests") {
     sources = [

media/BUILD.gn

@@ -610,15 +610,6 @@ if (is_ios) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("media_unittests_run") {
-  testonly = true
-  deps = [
-    ":media_unittests",
-  ]
-}
-
 test("media_unittests") {
   sources = [
     "capture/content/animated_content_sampler_unittest.cc",
@@ -839,15 +830,6 @@ test("media_perftests") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("audio_unittests_run") {
-  testonly = true
-  deps = [
-    ":audio_unittests",
-  ]
-}
-
 # For running the subset of media_unittests that might require audio hardware
 # separately on GPU bots. media_unittests includes these too.
 test("audio_unittests") {

media/blink/BUILD.gn

@@ -154,12 +154,3 @@ test("media_blink_unittests") {
 
   configs += [ "//v8:external_startup_data" ]
 }
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("media_blink_unittests_run") {
-  testonly = true
-  deps = [
-    ":media_blink_unittests",
-  ]
-}

media/cast/BUILD.gn

@@ -299,15 +299,6 @@ static_library("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("cast_unittests_run") {
-  testonly = true
-  deps = [
-    ":cast_unittests",
-  ]
-}
-
 test("cast_unittests") {
   sources = [
     "common/expanded_value_base_unittest.cc",

media/midi/BUILD.gn

@@ -150,15 +150,6 @@ component("midi") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("midi_unittests_run") {
-  testonly = true
-  deps = [
-    ":midi_unittests",
-  ]
-}
-
 test("midi_unittests") {
   sources = [
     "midi_manager_unittest.cc",

mojo/BUILD.gn

@@ -27,46 +27,6 @@ group("mojo") {
   deps += [ "//mojo/shell:all" ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("mojo_public_bindings_unittests_run") {
-  testonly = true
-  deps = [
-    # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_bindings_unittests"
-    "//third_party/mojo/src/mojo/edk/test:mojo_public_bindings_unittests",
-  ]
-}
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("mojo_public_environment_unittests_run") {
-  testonly = true
-  deps = [
-    # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_environment_unittests"
-    "//third_party/mojo/src/mojo/edk/test:mojo_public_environment_unittests",
-  ]
-}
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("mojo_public_system_unittests_run") {
-  testonly = true
-  deps = [
-    # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_system_unittests"
-    "//third_party/mojo/src/mojo/edk/test:mojo_public_system_unittests",
-  ]
-}
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("mojo_public_utility_unittests_run") {
-  testonly = true
-  deps = [
-    # TODO(use_chrome_edk): "//mojo/edk/test:mojo_public_utility_unittests"
-    "//third_party/mojo/src/mojo/edk/test:mojo_public_utility_unittests",
-  ]
-}
-
 group("tests") {
   testonly = true
   deps = [

mojo/common/BUILD.gn

@@ -64,15 +64,6 @@ source_set("url_type_converters") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("mojo_common_unittests_run") {
-  testonly = true
-  deps = [
-    ":mojo_common_unittests",
-  ]
-}
-
 # GYP version: mojo/mojo_base.gyp:mojo_common_unittests
 test("mojo_common_unittests") {
   deps = [

net/BUILD.gn

@@ -1369,15 +1369,6 @@ if (!is_ios) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("net_unittests_run") {
-  testonly = true
-  deps = [
-    ":net_unittests",
-  ]
-}
-
 test("net_unittests") {
   sources = gypi_values.net_test_sources
 

printing/BUILD.gn

@@ -196,15 +196,6 @@ component("printing") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("printing_unittests_run") {
-  testonly = true
-  deps = [
-    ":printing_unittests",
-  ]
-}
-
 test("printing_unittests") {
   sources = [
     "emf_win_unittest.cc",

remoting/BUILD.gn

@@ -104,15 +104,6 @@ source_set("test_support") {
 
 # TODO(GYP) remoting_unittests on Mac. Needs to be tested.
 if (!is_mac) {
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("remoting_unittests_run") {
-    testonly = true
-    deps = [
-      ":remoting_unittests",
-    ]
-  }
-
   test("remoting_unittests") {
     configs += [
       ":version",

sandbox/linux/BUILD.gn

@@ -192,7 +192,7 @@ action("bpf_dsl_golden") {
       rebase_path(outputs, root_build_dir) + rebase_path(inputs, root_build_dir)
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
+# TODO(GYP): Delete these after we've converted everything to GN.
 # The _run targets exist only for compatibility w/ GYP.
 group("sandbox_linux_unittests_run") {
   testonly = true
@@ -201,6 +201,15 @@ group("sandbox_linux_unittests_run") {
   ]
 }
 
+if (is_android) {
+  group("sandbox_linux_unittests_apk_run") {
+    testonly = true
+    deps = [
+      ":sandbox_linux_unittests",
+    ]
+  }
+}
+
 # The main sandboxing test target. "sandbox_linux_unittests" cannot use the
 # test() template because the test is run as an executable not as an APK on
 # Android.

skia/BUILD.gn

@@ -657,15 +657,6 @@ source_set("skia_opts") {
   visibility = [ ":skia" ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("skia_unittests_run") {
-  testonly = true
-  deps = [
-    ":skia_unittests",
-  ]
-}
-
 test("skia_unittests") {
   sources = [
     "ext/analysis_canvas_unittest.cc",

sql/BUILD.gn

@@ -69,15 +69,6 @@ source_set("redirection_header") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("sql_unittests_run") {
-  testonly = true
-  deps = [
-    ":sql_unittests",
-  ]
-}
-
 test("sql_unittests") {
   sources = [
     "connection_unittest.cc",

sync/BUILD.gn

@@ -610,15 +610,6 @@ static_library("test_support_sync_api") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("sync_unit_tests_run") {
-  testonly = true
-  deps = [
-    ":sync_unit_tests",
-  ]
-}
-
 # GYP version: sync/sync_tests.gypi:sync_unit_tests
 test("sync_unit_tests") {
   sources = [

testing/test.gni

@@ -186,6 +186,8 @@ template("test") {
       ]
     }
   } else {
+    main_target_name = target_name
+
     executable(target_name) {
       forward_variables_from(invoker, "*")
 
@@ -203,5 +205,14 @@ template("test") {
         "//build/win:default_exe_manifest",
       ]
     }
+
+    # TODO(GYP): Delete this after we've converted everything to GN.
+    # The _run targets exist only for compatibility with GYP.
+    group("${target_name}_run") {
+      testonly = true
+      deps = [
+        ":$main_target_name",
+      ]
+    }
   }
 }

third_party/WebKit/Source/platform/BUILD.gn

@@ -359,15 +359,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("blink_heap_unittests_run") {
-  testonly = true
-  deps = [
-    ":blink_heap_unittests",
-  ]
-}
-
 # GYP: blink_heap_unittests
 test("blink_heap_unittests") {
   visibility = []  # Allow re-assignment of list.
@@ -402,15 +393,6 @@ test("blink_heap_unittests") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("blink_platform_unittests_run") {
-  testonly = true
-  deps = [
-    ":blink_platform_unittests",
-  ]
-}
-
 test("blink_platform_unittests") {
   visibility = []  # Allow re-assignment of list.
   visibility = [ "*" ]

third_party/WebKit/Source/web/BUILD.gn

@@ -97,15 +97,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("webkit_unit_tests_run") {
-  testonly = true
-  deps = [
-    ":webkit_unit_tests",
-  ]
-}
-
 group("webkit_unit_tests_data") {
   data = [
     "tests/data/",

third_party/WebKit/Source/wtf/BUILD.gn

@@ -95,15 +95,6 @@ component("wtf") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("wtf_unittests_run") {
-  testonly = true
-  deps = [
-    ":wtf_unittests",
-  ]
-}
-
 test("wtf_unittests") {
   visibility = []  # Allow re-assignment of list.
   visibility = [ "*" ]

third_party/cacheinvalidation/BUILD.gn

@@ -90,15 +90,6 @@ static_library("cacheinvalidation") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("cacheinvalidation_unittests_run") {
-  testonly = true
-  deps = [
-    ":cacheinvalidation_unittests",
-  ]
-}
-
 test("cacheinvalidation_unittests") {
   sources = [
     "src/google/cacheinvalidation/impl/invalidation-client-impl_test.cc",

tools/battor_agent/BUILD.gn

@@ -59,12 +59,3 @@ test("battor_agent_unittests") {
     "//testing/gtest",
   ]
 }
-
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("battor_agent_unittests_run") {
-  testonly = true
-  deps = [
-    ":battor_agent_unittests",
-  ]
-}

tools/gn/BUILD.gn

@@ -241,15 +241,6 @@ executable("gn") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("gn_unittests_run") {
-  testonly = true
-  deps = [
-    ":gn_unittests",
-  ]
-}
-
 test("gn_unittests") {
   sources = [
     "action_target_generator_unittest.cc",

ui/accessibility/BUILD.gn

@@ -126,15 +126,6 @@ source_set("test_support") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("accessibility_unittests_run") {
-  testonly = true
-  deps = [
-    ":accessibility_unittests",
-  ]
-}
-
 test("accessibility_unittests") {
   sources = [
     "ax_generated_tree_unittest.cc",

ui/app_list/BUILD.gn

@@ -267,15 +267,6 @@ if (is_linux && !is_chromeos) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("app_list_unittests_run") {
-  testonly = true
-  deps = [
-    ":app_list_unittests",
-  ]
-}
-
 test("app_list_unittests") {
   sources = [
     "app_list_item_list_unittest.cc",

ui/aura/BUILD.gn

@@ -279,15 +279,6 @@ executable("bench") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("aura_unittests_run") {
-  testonly = true
-  deps = [
-    ":aura_unittests",
-  ]
-}
-
 test("aura_unittests") {
   sources = [
     "gestures/gesture_recognizer_unittest.cc",

ui/base/BUILD.gn

@@ -661,16 +661,6 @@ if (is_android) {
   }
 }
 
-#
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("ui_base_unittests_run") {
-  testonly = true
-  deps = [
-    ":ui_base_unittests",
-  ]
-}
-
 # GYP version: ui/base/ui_base_tests.gyp:ui_base_unittests
 test("ui_base_unittests") {
   sources = [

ui/compositor/BUILD.gn

@@ -160,15 +160,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("compositor_unittests_run") {
-  testonly = true
-  deps = [
-    ":compositor_unittests",
-  ]
-}
-
 test("compositor_unittests") {
   sources = [
     "callback_layer_animation_observer_unittest.cc",

ui/display/BUILD.gn

@@ -130,15 +130,6 @@ if (is_chromeos) {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("display_unittests_run") {
-  testonly = true
-  deps = [
-    ":display_unittests",
-  ]
-}
-
 # This test covers the ChromeOS "display" target as well as the cross-platform
 # //display/util target.
 test("display_unittests") {

ui/events/BUILD.gn

@@ -333,15 +333,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("events_unittests_run") {
-  testonly = true
-  deps = [
-    ":events_unittests",
-  ]
-}
-
 test("events_unittests") {
   sources = [
     "android/scroller_unittest.cc",

ui/gfx/BUILD.gn

@@ -627,15 +627,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("gfx_unittests_run") {
-  testonly = true
-  deps = [
-    ":gfx_unittests",
-  ]
-}
-
 test("gfx_unittests") {
   sources = [
     "font_render_params_linux_unittest.cc",

ui/gl/BUILD.gn

@@ -328,15 +328,6 @@ source_set("test_support") {
   }
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("gl_unittests_run") {
-  testonly = true
-  deps = [
-    ":gl_unittests",
-  ]
-}
-
 test("gl_unittests") {
   sources = [
     "gl_api_unittest.cc",

ui/touch_selection/BUILD.gn

@@ -68,15 +68,6 @@ static_library("test_support") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("ui_touch_selection_unittests_run") {
-  testonly = true
-  deps = [
-    ":ui_touch_selection_unittests",
-  ]
-}
-
 test("ui_touch_selection_unittests") {
   sources = [
     "longpress_drag_selector_unittest.cc",

ui/wm/BUILD.gn

@@ -107,15 +107,6 @@ static_library("test_support") {
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("wm_unittests_run") {
-  testonly = true
-  deps = [
-    ":wm_unittests",
-  ]
-}
-
 test("wm_unittests") {
   sources = [
     "core/capture_controller_unittest.cc",

url/BUILD.gn

@@ -115,15 +115,6 @@ if (is_android) {
 
 # TODO(dpranke): crbug.com/360936. Get this to build and run on Android.
 if (!is_android) {
-  # TODO(GYP): Delete this after we've converted everything to GN.
-  # The _run targets exist only for compatibility w/ GYP.
-  group("url_unittests_run") {
-    testonly = true
-    deps = [
-      ":url_unittests",
-    ]
-  }
-
   test("url_unittests") {
     sources = [
       "gurl_unittest.cc",