🚀 Geneate base_unittest's Android .isolate file at build-time

testing/test.gni

 # Copyright 2015 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.
 
 # ==============================================================================
 # TEST SETUP
 # ==============================================================================
 
+if (is_android) {
+  template("_gen_apk_isolate") {
+    action(target_name) {
+      script = "//build/android/gyp/gen-android-test-isolate.py"
+      outputs = [
+        invoker.output,
+      ]
+      args = [
+        "--out-file",
+        rebase_path(invoker.output, root_build_dir),
+        ".",
+        invoker.lib_target,
+      ]
+    }
+  }
+}
+
 # Define a test as an executable (or apk on Android) with the "testonly" flag
 # set.
+#
+# Variables:
+#   requires_apk_isolate: Whether the test has files (via data / data_deps) that
+#       need to be side-loaded to the device.
 template("test") {
+  _requires_apk_isolate =

[jbudorick, 2016/03/15 20:23:14]

Why define this outside of is_android?

[agrieve, 2016/03/15 20:34:07]

I thought it'd be nice to have it live right beside data = in the callers. I
have to access it here to mark it as used for non-android platforms.

Definitely don't feel strongly about this though, and I think we can get rid of
the variable once gn supports a primitive for writing the deps (we'll just write
them for every test and ignore it if it's empty)

+      defined(invoker.requires_apk_isolate) && invoker.requires_apk_isolate
+  assert(_requires_apk_isolate || true)  # Mark as used.
+
   if (is_android) {
     import("//build/config/android/config.gni")
     import("//build/config/android/rules.gni")
 
     main_target_name = target_name
     library_name = "_${target_name}__library"
     apk_name = "${target_name}_apk"
 
+    if (_requires_apk_isolate) {
+      _apk_isolate_target_name = "_${target_name}__apk_isolate"
+      _apk_isolate_path = "$target_gen_dir/$target_name.apk.isolate"
+      _gen_apk_isolate(_apk_isolate_target_name) {
+        lib_target = get_label_info(":$library_name", "label_no_toolchain")
+        output = _apk_isolate_path
+      }
+    }
     shared_library(library_name) {
       # Configs will always be defined since we set_defaults for a component
       # in the main config. We want to use those rather than whatever came with
       # the nested shared/static library inside the component.
       configs = []  # Prevent list overwriting warning.
       configs = invoker.configs
 
       testonly = true
 
       # Don't use "*" to forward all variables since some (like output_name
@@ skipping 52 matching lines @@
     _test_name = main_target_name
     if (defined(invoker.output_name)) {
       _test_name = invoker.output_name
     }
     test_runner_script_name = "${_test_name}__test_runner_script"
     test_runner_script(test_runner_script_name) {
       apk_target = ":$apk_name"
       test_name = _test_name
       test_type = "gtest"
       test_suite = _test_name
-      if (defined(invoker.isolate_file)) {
+      if (_requires_apk_isolate) {
+        isolate_file = _apk_isolate_path
+        # TODO(agrieve): Delete next conditional once it's not used.
+      } else if (defined(invoker.isolate_file)) {
         isolate_file = invoker.isolate_file
       }
     }
     incremental_test_runner_script_name =
         "${_test_name}_incremental__test_runner_script"
     test_runner_script(incremental_test_runner_script_name) {
       apk_target = ":$apk_name"
       test_name = "${_test_name}_incremental"
       test_type = "gtest"
       test_suite = _test_name
       incremental_install = true
-      if (defined(invoker.isolate_file)) {
+      if (_requires_apk_isolate) {
+        isolate_file = _apk_isolate_path
+        # TODO(agrieve): Delete next conditional once it's not used.
+      } else if (defined(invoker.isolate_file)) {
         isolate_file = invoker.isolate_file
       }
     }
 
     group(target_name) {
       testonly = true
-      datadeps = [
+      data_deps = [
         ":$test_runner_script_name",
       ]
+      if (_requires_apk_isolate) {
+        data_deps += [ ":$_apk_isolate_target_name" ]
+      }
       deps = [
         ":$apk_name",
       ]
     }
     group("${target_name}_incremental") {
       testonly = true
-      datadeps = [
+      data_deps = [
         ":$incremental_test_runner_script_name",
       ]
+      if (_requires_apk_isolate) {
+        data_deps += [ ":$_apk_isolate_target_name" ]
+      }
       deps = [
         ":${apk_name}_incremental",
       ]
     }
 
     # TODO(GYP): Delete this after we've converted everything to GN.
     # The _run targets exist only for compatibility w/ GYP.
     group("${target_name}_apk_run") {
       testonly = true
       deps = [
@@ skipping 76 matching lines @@
     # 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",
       ]
     }
   }
 }