[Android] Fix generated scripts for junit tests.

build/config/android/internal_rules.gni

Index: build/config/android/internal_rules.gni
diff --git a/build/config/android/internal_rules.gni b/build/config/android/internal_rules.gni
index 7e279c3ee8fbf07e683044d232165f04ce914122..8a421bea768a7fb1d8cbca73648ecb04daffb1d4 100644
--- a/build/config/android/internal_rules.gni
+++ b/build/config/android/internal_rules.gni
@@ -198,7 +198,7 @@ template("java_binary_script") {
   action(target_name) {
     script = "//build/android/gyp/create_java_binary_script.py"
     depfile = "$target_gen_dir/$_script_name.d"
-    java_script = "$root_build_dir/bin/$_script_name"
+    java_script = "$root_build_dir/bin/helper/$_script_name"
     inputs = [
       _build_config,
     ]
@@ -1294,7 +1294,11 @@ template("java_prebuilt_impl") {
                              ])
       build_config = _build_config
       jar_path = _jar_path
-      script_name = _template_name
+      if (defined(invoker.wrapper_script_name)) {

[jbudorick, 2016/02/09 00:06:47]

Why are these overrides suddenly necessary?

[mikecase (-- gone --), 2016/02/09 00:54:34]

Unfortunately, because of..
https://code.google.com/p/chromium/codesearch#chromium/src/build/android/pyli...

The way the test runner runs the junit test it runs something named
out/bin/helper/<suite_name>
and then looks for a <suite_name>.jar. Without these two overrides, the file is
named <suite_name>__java_binary and the jar is named
<suite_name>__java_binary.jar and the test_runner
doesnt see them.

+        script_name = invoker.wrapper_script_name
+      } else {
+        script_name = _template_name
+      }
       deps = [
         ":$_build_config_target_name",
       ]
@@ -1532,8 +1536,13 @@ template("java_library_impl") {
 
   # Jar files can be needed at runtime (by Robolectric tests or java binaries),
   # so do not put them under gen/.
+  if (defined(invoker.jar_name)) {
+    _jar_name = invoker.jar_name
+  } else {
+    _jar_name = target_name
+  }

[mikecase (-- gone --), 2016/02/09 00:54:34]

This is necessary unfortunately because the test_runner.py looks for
<suite_name>.jar.
Without this override the jar is named <suite_name>__java_binary.jar and the
test_runner doesnt
see it

   target_dir_name = get_label_info(":$target_name", "dir")
-  _jar_path = "$root_out_dir/lib.java$target_dir_name/$target_name.jar"
+  _jar_path = "$root_out_dir/lib.java$target_dir_name/$_jar_name.jar"
   if (defined(invoker.jar_path)) {
     _jar_path = invoker.jar_path
   }
@@ -1687,7 +1696,11 @@ template("java_library_impl") {
                              ])
       build_config = _build_config
       jar_path = _jar_path
-      script_name = _template_name
+      if (defined(invoker.wrapper_script_name)) {
+        script_name = invoker.wrapper_script_name
+      } else {
+        script_name = _template_name
+      }
       deps = build_config_deps
     }
   }
@@ -2069,6 +2082,12 @@ template("test_runner_script") {
           rebase_path("$root_out_dir/coverage", root_build_dir),
         ]
       }
+    } else if (_test_type == "junit") {
+      assert(defined(invoker.test_suite))
+      test_runner_args += [
+        "--test-suite",
+        invoker.test_suite,
+      ]
     } else {
       assert(false, "Invalid test type: $_test_type.")
     }