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

Unified Diff: content/content_shell.gypi

Issue 12047068: Add apk for running content_browsertests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments Created 7 years, 11 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 | « content/content.gyp ('k') | content/content_tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/content_shell.gypi
diff --git a/content/content_shell.gypi b/content/content_shell.gypi
index a977e2760668ab8722c9c4fa2b359345eb22caa9..9e59c7d86fd260733bc26c1d0f7b63c70d2cef4a 100644
--- a/content/content_shell.gypi
+++ b/content/content_shell.gypi
@@ -53,6 +53,10 @@
'..',
],
'sources': [
+ 'shell/android/shell_jni_registrar.cc',
+ 'shell/android/shell_jni_registrar.h',
+ 'shell/android/shell_manager.cc',
+ 'shell/android/shell_manager.h',
'shell/geolocation/shell_access_token_store.cc',
'shell/geolocation/shell_access_token_store.h',
'shell/minimal_ash.cc',
@@ -556,6 +560,7 @@
'target_name': 'content_shell_jni_headers',
'type': 'none',
'sources': [
+ 'shell/android/browsertests_apk/src/org/chromium/content_browsertests_apk/ContentBrowserTestsActivity.java',
'shell/android/java/src/org/chromium/content_shell/ShellManager.java',
'shell/android/java/src/org/chromium/content_shell/Shell.java',
],
@@ -584,12 +589,6 @@
'sources': [
'shell/android/shell_library_loader.cc',
'shell/android/shell_library_loader.h',
- 'shell/android/shell_manager.cc',
- 'shell/android/shell_manager.h',
- ],
- 'sources!': [
- 'shell/shell_main.cc',
- 'shell/shell_main.h',
],
'conditions': [
['android_build_type==1', {
@@ -600,11 +599,27 @@
],
},
{
+ 'target_name': 'content_shell_java',
+ 'type': 'none',
+ 'dependencies': [
+ 'content_java',
+ ],
+ 'variables': {
+ 'package_name': 'content_shell',
+ 'java_in_dir': '../content/shell/android/java',
+ 'has_java_resources': 1,
+ 'R_package': 'org.chromium.content_shell',
+ 'R_package_relpath': 'org/chromium/content_shell',
+ },
+ 'includes': [ '../build/java.gypi' ],
+ },
+ {
# content_shell_apk creates a .jar as a side effect. Any java targets
# that need that .jar in their classpath should depend on this target,
- # content_shell_java. Dependents of content_shell_apk receive its jar
- # path in the variable 'apk_output_jar_path'.
- 'target_name': 'content_shell_java',
+ # content_shell_apk_java. Dependents of content_shell_apk receive its
+ # jar path in the variable 'apk_output_jar_path'. This target should
+ # only be used by targets which instrument content_shell_apk.
+ 'target_name': 'content_shell_apk_java',
'type': 'none',
'dependencies': [
'content_shell_apk',
@@ -633,21 +648,22 @@
'type': 'none',
'dependencies': [
'content_java',
+ 'content_shell_java',
+ 'libcontent_shell_content_view',
'../base/base.gyp:base_java',
'../media/media.gyp:media_java',
'../net/net.gyp:net_java',
'../ui/ui.gyp:ui_java',
],
'variables': {
- 'package_name': 'content_shell',
+ 'package_name': 'content_shell_apk',
'apk_name': 'ContentShell',
- 'manifest_package_name': 'org.chromium.content_shell',
- 'java_in_dir': 'shell/android/java',
- # TODO(cjhopman): The resource directory of all apks should be in
- # <java_in_dir>/res.
- 'resource_dir': '../res',
+ 'manifest_package_name': 'org.chromium.content_shell_apk',
+ 'java_in_dir': 'shell/android/shell_apk',
+ 'resource_dir': 'res',
'native_libs_paths': ['<(SHARED_LIB_DIR)/libcontent_shell_content_view.so'],
'additional_input_paths': ['<(PRODUCT_DIR)/content_shell/assets/content_shell.pak'],
+ 'asset_location': '<(ant_build_out)/content_shell/assets',
},
'includes': [ '../build/java_apk.gypi' ],
},
« no previous file with comments | « content/content.gyp ('k') | content/content_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698