Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2787)

Unified Diff: build/config/android/internal_rules.gni

Issue 1208483004: Make instrumentation test dependency on a support APK explicit (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: make paths relative to the output directory Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/config/android/internal_rules.gni
diff --git a/build/config/android/internal_rules.gni b/build/config/android/internal_rules.gni
index 091ee4ac2ebc7051e855c45ecba0a2fb7392d1f3..4166978916c03401b68ac01456839dce6e90b3ed 100644
--- a/build/config/android/internal_rules.gni
+++ b/build/config/android/internal_rules.gni
@@ -1514,6 +1514,14 @@ template("test_runner_script") {
"--test-apk",
invoker.test_apk,
]
+ if (defined(invoker.support_apk_name)) {
+ _support_apk_name = invoker.support_apk_name
+
+ test_runner_args += [
+ "--support-apk",
+ "apks/$_support_apk_name.apk",
dgn 2015/06/24 21:29:50 I tried to take a target (e.g. "//foo:foo_support_
jbudorick 2015/06/25 02:59:59 I'm not that familiar with GN idioms, so +cjhopman
cjhopman 2015/06/30 23:07:13 Yeah, can't really get the output from a label. Th
+ ]
+ }
} else {
assert(false, "Invalid test type: $_test_type.")
}

Powered by Google App Engine
This is Rietveld 408576698