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

Unified Diff: components/components_tests.gyp

Issue 1143903002: [Android] Refactor the native test wrappers. (RELAND) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 7 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
« no previous file with comments | « components/BUILD.gn ('k') | components/test/android/browsertests_apk/AndroidManifest.xml.jinja2 » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/components_tests.gyp
diff --git a/components/components_tests.gyp b/components/components_tests.gyp
index 623dd94812b348fe5e554746a33f25fbf8bedb26..00d8805384574fe98cc49f433153db07143ee840 100644
--- a/components/components_tests.gyp
+++ b/components/components_tests.gyp
@@ -1209,17 +1209,6 @@
'includes': [ '../build/android/jinja_template.gypi' ],
},
{
- 'target_name': 'components_browsertests_jni_headers',
- 'type': 'none',
- 'sources': [
- 'test/android/browsertests_apk/src/org/chromium/components_browsertests_apk/ComponentsBrowserTestsActivity.java',
- ],
- 'variables': {
- 'jni_gen_package': 'content/shell',
- },
- 'includes': [ '../build/jni_generator.gypi' ],
- },
- {
# TODO(GN)
'target_name': 'components_browsertests_apk',
'type': 'none',
@@ -1233,14 +1222,13 @@
'components_browsertests',
],
'variables': {
- 'apk_name': 'components_browsertests',
+ 'test_suite_name': 'components_browsertests',
'java_in_dir': 'test/android/browsertests_apk',
'android_manifest_path': '<(SHARED_INTERMEDIATE_DIR)/components_browsertests_manifest/AndroidManifest.xml',
'resource_dir': 'test/android/browsertests_apk/res',
- 'native_lib_target': 'libcomponents_browsertests',
'asset_location': '<(PRODUCT_DIR)/components_browsertests_apk_shell/assets',
},
- 'includes': [ '../build/java_apk.gypi' ],
+ 'includes': [ '../build/apk_browsertest.gypi' ],
},
],
}],
@@ -1326,16 +1314,13 @@
'conditions': [
['OS == "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',
],
'dependencies': [
- '../testing/android/native_test.gyp:native_test_util',
- 'components_browsertests_jni_headers',
+ '../testing/android/native_test.gyp:native_test_support',
],
}],
['OS == "linux"', {
« no previous file with comments | « components/BUILD.gn ('k') | components/test/android/browsertests_apk/AndroidManifest.xml.jinja2 » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698