👀 Make sure Android GN instrumentation test targets expose a _run alias

build/config/android/rules.gni

 # 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("//base/android/linker/config.gni")
 import("//build/config/android/config.gni")
 import("//build/config/android/internal_rules.gni")
 import("//build/config/sanitizers/sanitizers.gni")
 import("//build/toolchain/toolchain.gni")
 import("//third_party/android_platform/config.gni")
@@ skipping 2037 matching lines @@
       ":$_apk_target_name",
       ":$_test_runner_target_name",
 
       # Required by test runner to enumerate test list.
       ":${_apk_target_name}_dist_ijar",
     ]
     if (defined(invoker.apk_under_test)) {
       public_deps += [ invoker.apk_under_test ]
     }
   }
+
+  # TODO(GYP): Delete once recipes no longer use this target.
+  group("${target_name}_run") {
+    public_deps = [
+      ":${invoker.target_name}",

[jbudorick, 2016/04/27 15:13:46]

I'm assuming this gets the right target name (i.e., the one without _run)
because of the 'invoker.'?

[agrieve, 2016/04/27 15:25:37]

it does :)

+    ]
+  }
   group("${target_name}_incremental") {
     public_deps = [
       ":${_apk_target_name}_dist_ijar",
       ":${_apk_target_name}_incremental",
       ":${_test_runner_target_name}_incremental",
     ]
     if (defined(invoker.apk_under_test)) {
       public_deps += [ "${invoker.apk_under_test}_incremental" ]
     }
   }
@@ skipping 338 matching lines @@
       "--target",
       rebase_path(invoker.target, root_build_dir),
       "--output-directory",
       rebase_path(root_out_dir, root_build_dir),
     ]
     if (defined(invoker.flag_name)) {
       args += [ "--flag-name=${invoker.flag_name}" ]
     }
   }
 }