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

Unified Diff: build/android/test_runner.gypi

Issue 1208483004: Make instrumentation test dependency on a support APK explicit (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use GYP and GN to set whether the support apk is needed 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/android/test_runner.gypi
diff --git a/build/android/test_runner.gypi b/build/android/test_runner.gypi
index 92f355bf1fb4abe73d3480f728ac5f7ecfd7665b..e5bbfb829fd79b66e0789e2f2b5dfb804e7467cd 100644
--- a/build/android/test_runner.gypi
+++ b/build/android/test_runner.gypi
@@ -33,6 +33,7 @@
'variables': {
'variables': {
'isolate_file%': '',
+ 'support_apk_name%': '',
},
'test_runner_args': ['--output-directory', '<(PRODUCT_DIR)'],
'conditions': [
@@ -43,6 +44,11 @@
['test_type == "instrumentation"', {
'test_runner_args': ['--test-apk', '<(apk_name)'],
'script_name': 'run_<(_target_name)',
+ 'conditions': [
+ ['support_apk_name != ""', {
+ 'test_runner_args+': ['--support-apk', '<(support_apk_name)'],
+ }],
+ ],
}],
['isolate_file != ""', {
'test_runner_args': ['--isolate-file-path', '<(isolate_file)']
dgn 2015/06/24 16:03:04 My change works but I don't quite understand how i
jbudorick 2015/06/24 17:46:39 gyp seems to append to lists even if + isn't inclu

Powered by Google App Engine
This is Rietveld 408576698