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

Unified Diff: components/BUILD.gn

Issue 1075783002: Enable components_browsertests on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix GN build and add java dependency to components/test/DEPS Created 5 years, 8 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: components/BUILD.gn
diff --git a/components/BUILD.gn b/components/BUILD.gn
index 95761c19b55c248fb561256c7434602f785e51b7..01503dc10791a29d1dbe5421351c7a69a687f8f0 100644
--- a/components/BUILD.gn
+++ b/components/BUILD.gn
@@ -334,6 +334,17 @@ repack("components_tests_pak") {
]
}
+if (is_android) {
+ import("//build/config/android/rules.gni")
+
+ generate_jni("components_browsertests_jni_headers") {
+ jni_package = "components_browsertests/shell"
+ sources = [
+ "test/android/browsertests_apk/src/org/chromium/components_browsertests_apk/ComponentsBrowserTestsActivity.java",
+ ]
+ }
+}
+
test("components_browsertests") {
sources = [
"autofill/content/browser/risk/fingerprint_browsertest.cc",
@@ -369,7 +380,18 @@ test("components_browsertests") {
data_deps = [ ":components_tests_pak" ]
if (is_android) {
+ sources += [
+ "test/android/browsertests_apk/components_browser_tests_android.cc",
+ "test/android/browsertests_apk/components_browser_tests_android.h",
+ "test/android/browsertests_apk/components_browser_tests_jni_onload.cc",
+ ]
sources -= [ "autofill/content/browser/risk/fingerprint_browsertest.cc" ]
+ deps += [
+ ":components_browsertests_jni_headers",
+ "//testing/android/native_test:native_test_util",
+ ]
+
+ use_launcher = false
}
if (is_linux) {

Powered by Google App Engine
This is Rietveld 408576698