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

Unified Diff: testing/test.gni

Issue 2549363004: Multiprocess test client: Android child process launcher rework. (Closed)
Patch Set: Addressed comments + synced Created 4 years 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: testing/test.gni
diff --git a/testing/test.gni b/testing/test.gni
index b6677d45fc46d62ab61b6e78be6b66f5ddab9a5f..f93668ddf22b376a9e169ecb72856306fb9cb0fa 100644
--- a/testing/test.gni
+++ b/testing/test.gni
@@ -6,6 +6,12 @@
# TEST SETUP
# ==============================================================================
+if (is_android) {
+ import("//build/config/android/config.gni")
+ import("//build/config/android/rules.gni")
+ import("//build/config/sanitizers/sanitizers.gni")
+}
+
# Define a test as an executable (or apk on Android) with the "testonly" flag
# set.
# Variable:
@@ -13,10 +19,6 @@
# use_native_activity: Test implements ANativeActivity_onCreate().
template("test") {
if (is_android) {
- import("//build/config/android/config.gni")
- import("//build/config/android/rules.gni")
- import("//build/config/sanitizers/sanitizers.gni")
-
_use_raw_android_executable = defined(invoker.use_raw_android_executable) &&
invoker.use_raw_android_executable
@@ -115,6 +117,9 @@ template("test") {
install_script_name = "install_${invoker.output_name}"
}
+ # Add the Java classes so that each target does not have to do it.
+ deps += [ "//base/test:test_support_java" ]
+
# 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

Powered by Google App Engine
This is Rietveld 408576698