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

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

Issue 1465383003: [Android] Add ChromiumNetTestSupport.apk for the java EmbeddedTestServer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: +landmine for GN Created 5 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: build/config/android/rules.gni
diff --git a/build/config/android/rules.gni b/build/config/android/rules.gni
index aeb7f1e4aadde2e94c70e3bf31b865f2f5db63de..5af8ee23c8b9c866b7551d8c0b542ff41fa75c98 100644
--- a/build/config/android/rules.gni
+++ b/build/config/android/rules.gni
@@ -1351,6 +1351,17 @@ template("android_apk") {
if (defined(invoker.native_lib_version_arg)) {
_native_lib_version_arg = invoker.native_lib_version_arg
}
+
+ _native_libraries_template =
+ "//base/android/java/templates/NativeLibraries.template"
+ if (defined(invoker.native_libraries_template)) {
+ _native_libraries_template = invoker.native_libraries_template
+ }
+
+ _native_libraries_package_name = "org/chromium/base/library_loader"
+ if (defined(invoker.native_libraries_package_name)) {
+ _native_libraries_package_name = invoker.native_libraries_package_name
+ }
}
_android_manifest_deps = []
@@ -1436,9 +1447,9 @@ template("android_apk") {
}
java_cpp_template("${_template_name}__native_libraries_java") {
- package_name = "org/chromium/base/library_loader"
+ package_name = _native_libraries_package_name
sources = [
- "//base/android/java/templates/NativeLibraries.template",
+ _native_libraries_template,
]
inputs = [
_build_config,
@@ -1466,6 +1477,22 @@ template("android_apk") {
}
}
_srcjar_deps += [ ":${_template_name}__native_libraries_java" ]
+ } else if (defined(invoker.native_libraries_template) &&
+ invoker.native_libraries_template != "") {
+ java_cpp_template("${_template_name}__empty_native_libraries_java") {
+ assert(defined(invoker.native_libraries_package_name))
+ package_name = invoker.native_libraries_package_name
+ sources = [
+ invoker.native_libraries_template,
+ ]
+ inputs = [
+ _build_config,
+ ]
+ deps = [
+ ":$build_config_target",
+ ]
+ }
+ _srcjar_deps += [ ":${_template_name}__empty_native_libraries_java" ]
}
java_target = "${_template_name}__java"
@@ -1910,13 +1937,20 @@ template("instrumentation_test_apk") {
deps = []
data_deps = []
forward_variables_from(invoker, "*")
- deps += [ "//testing/android/broker:broker_java" ]
+ deps += [
+ "//base:base_java",
+ "//testing/android/broker:broker_java",
+ ]
data_deps += [
"//testing/android/driver:driver_apk",
"//tools/android/forwarder2",
"//tools/android/md5sum",
]
+ native_libraries_template =
+ "//base/test/android/java/templates/NativeTestLibraries.template"
+ native_libraries_package_name = "org.chromium.base.test.library_loader"
+
run_findbugs =
defined(invoker.run_findbugs) && invoker.run_findbugs &&
(defined(invoker.java_files) || defined(invoker.DEPRECATED_java_in_dir))

Powered by Google App Engine
This is Rietveld 408576698