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

Unified Diff: base/test/BUILD.gn

Issue 2549363004: Multiprocess test client: Android child process launcher rework. (Closed)
Patch Set: Sublime ate my channel_pair change... 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: base/test/BUILD.gn
diff --git a/base/test/BUILD.gn b/base/test/BUILD.gn
index 8df234189abc1ea69425e6bae08c3e5f88f69fb4..bc16bc637cc41bff4cf50efd9c3c8f21ba53c857 100644
--- a/base/test/BUILD.gn
+++ b/base/test/BUILD.gn
@@ -22,12 +22,25 @@ static_library("test_config") {
]
}
+# This target is needed for components on Android depending on base:test_support
+# that do not define a main function. This is because multiprocess support on
+# Android requires a main function (the one invoked on the child process).
+static_library("test_support_main_stub") {
+ testonly = true
+ sources = [
+ "main_stub.cc",
+ ]
+}
+
static_library("test_support") {
testonly = true
sources = [
"../trace_event/trace_config_memory_test_util.h",
"android/java_handler_thread_for_testing.cc",
"android/java_handler_thread_for_testing.h",
+ "android/multiprocess_test_client_service.cc",
+ "android/test_support_jni_registrar.cc",
+ "android/test_support_jni_registrar.h",
"android/test_system_message_handler_link_android.cc",
"android/test_system_message_handler_link_android.h",
"fuzzed_data_provider.cc",
@@ -56,7 +69,9 @@ static_library("test_support") {
"mock_entropy_provider.h",
"mock_log.cc",
"mock_log.h",
+ "multiprocess_test.cc",
"multiprocess_test.h",
+ "multiprocess_test_android.cc",
"null_task_runner.cc",
"null_task_runner.h",
"opaque_ref_counted.cc",
@@ -146,8 +161,6 @@ static_library("test_support") {
"launcher/test_launcher_tracer.h",
"launcher/test_results_tracker.cc",
"launcher/unit_test_launcher.cc",
- "multiprocess_test.cc",
- "multiprocess_test_android.cc",
]
}
@@ -191,7 +204,11 @@ static_library("test_support") {
}
if (is_android) {
- deps += [ ":base_unittests_jni_headers" ]
+ deps += [
+ ":base_unittests_jni_headers",
+ ":test_support_jni_headers",
+ ]
+ public_deps += [ ":test_support_java" ]
}
if (is_nacl_nonsfi) {
@@ -327,4 +344,40 @@ if (is_android) {
]
jni_package = "base"
}
+
+ generate_jni("test_support_jni_headers") {
+ sources = [
+ "android/java/src/org/chromium/base/MainReturnCodeResult.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientLauncher.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientService.java",
+ ]
+ jni_package = "base"
+ }
+
+ android_library("test_support_java") {
+ testonly = true
+ deps = [
+ "//base:base_java",
+ ]
+ srcjar_deps = [ ":test_support_java_aidl" ]
+ java_files = [
+ "android/java/src/org/chromium/base/FileDescriptorInfo.java",
+ "android/java/src/org/chromium/base/MainReturnCodeResult.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientLauncher.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientService.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientService0.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientService1.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientService2.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientService3.java",
+ "android/java/src/org/chromium/base/MultiprocessTestClientService4.java",
+ ]
+ }
+
+ android_aidl("test_support_java_aidl") {
+ testonly = true
+ import_include = "android/java/src"
+ sources = [
+ "android/java/src/org/chromium/base/ITestClient.aidl",
+ ]
+ }
}

Powered by Google App Engine
This is Rietveld 408576698