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

Unified Diff: components/components_tests.gyp

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/components_tests.gyp
diff --git a/components/components_tests.gyp b/components/components_tests.gyp
index 64cdc9d0a86876dc8f3769c771263fd73053083d..f48690f0033e77a182f70d17ad2e885f19e81053 100644
--- a/components/components_tests.gyp
+++ b/components/components_tests.gyp
@@ -1123,6 +1123,87 @@
},
],
'conditions': [
+ ['OS == "android"', {
+ 'variables': {
+ 'components_browsertests_pak_input_resources': [
+ '<(PRODUCT_DIR)/components_tests_resources.pak',
+ '<(PRODUCT_DIR)/content_shell/assets/content_shell.pak',
+ ],
+ 'conditions': [
+ ['icu_use_data_file_flag==1', {
+ 'components_browsertests_pak_input_resources': [
+ '<(PRODUCT_DIR)/icudtl.dat',
+ ],
+ }],
+ ['v8_use_external_startup_data==1', {
+ 'components_browsertests_pak_input_resources': [
+ '<(PRODUCT_DIR)/natives_blob.bin',
+ '<(PRODUCT_DIR)/snapshot_blob.bin',
+ ],
+ }],
+ ],
+ },
+ 'targets': [
+ {
+ 'target_name': 'components_browsertests_paks_copy',
+ 'type': 'none',
+ 'dependencies': [
+ 'components_browsertests',
+ ],
+ 'copies': [
+ {
+ 'destination': '<(PRODUCT_DIR)/components_browsertests_apk_shell/assets',
+ 'files': [
+ '<@(components_browsertests_pak_input_resources)',
+ ],
+ }
+ ],
+ },
+ {
+ 'target_name': 'components_browsertests_manifest',
+ 'type': 'none',
+ 'variables': {
+ 'jinja_inputs': ['test/android/browsertests_apk/AndroidManifest.xml.jinja2'],
+ 'jinja_output': '<(SHARED_INTERMEDIATE_DIR)/components_browsertests_manifest/AndroidManifest.xml',
+ },
+ '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',
+ 'dependencies': [
+ '../content/content.gyp:content_icudata',
+ '../content/content.gyp:content_java',
+ '../content/content.gyp:content_v8_external_data',
+ '../content/content_shell_and_tests.gyp:content_java_test_support',
+ '../content/content_shell_and_tests.gyp:content_shell_java',
+ 'components_browsertests_paks_copy',
+ 'components_browsertests',
+ ],
+ 'variables': {
+ 'apk_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' ],
+ },
+ ],
+ }],
['OS != "ios"', {
'targets': [
{
@@ -1204,9 +1285,18 @@
],
'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',
+ ],
}],
['OS == "linux"', {
'sources': [
« no previous file with comments | « components/components_browsertests.isolate ('k') | components/dom_distiller/content/distillable_page_utils_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698