Android: Address old TODO about removing a data_dep
Also renames the nested targets to follow current conventions and to
ensure that any bot that does still depend on the _apk target will fail
because of this.
TBR=agrieve # trivial test update
Change-Id: Ic958e1e74dd2f9628512369bd5a7ce01dd9c19cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1857571
Commit-Queue: Andrew Grieve <[email protected]>
Reviewed-by: John Budorick <[email protected]>
Cr-Commit-Position: refs/heads/master@{#707850}
diff --git a/testing/test.gni b/testing/test.gni
index 8882424..654cea2 100644
--- a/testing/test.gni
+++ b/testing/test.gni
@@ -90,8 +90,8 @@
}
}
} else {
- _library_target = "_${target_name}__library"
- _apk_target = "${target_name}_apk"
+ _library_target = "${target_name}__library"
+ _apk_target = "${target_name}__apk"
_apk_specific_vars = [
"android_manifest",
"android_manifest_dep",
@@ -160,16 +160,6 @@
if (defined(_unwind_table_asset_name)) {
deps += [ ":${_unwind_table_asset_name}" ]
}
-
- # TODO(agrieve): Remove this data_dep once bots don't build the _apk
- # target (post-GYP).
- # It's a bit backwards for the apk to depend on the runner script, since
- # the apk is conceptually a runtime_dep of the script. However, it is
- # currently necessary because the bots build this _apk target directly
- # rather than the group() below.
- data_deps = [
- ":$_test_runner_target",
- ]
}
_test_runner_target = "${_output_name}__test_runner_script"
