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

Unified Diff: chrome/chrome_android.gypi

Issue 10943014: Add support for the ChromiumTestShellTest APK. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed commented out gyp actions. Created 8 years, 3 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: chrome/chrome_android.gypi
diff --git a/chrome/chrome_android.gypi b/chrome/chrome_android.gypi
index 3fc4bd7d3911d19e89b5c3d978d0227561114cb8..327e32654d5fa34c096e4244c43c1894328ff88c 100644
--- a/chrome/chrome_android.gypi
+++ b/chrome/chrome_android.gypi
@@ -39,9 +39,22 @@
'target_name': 'chromium_testshell',
'type': 'none',
'dependencies': [
+ '../base/base.gyp:base_java',
+ '../content/content.gyp:content_java',
+ '../media/media.gyp:media_java',
Yaron 2012/09/20 03:05:17 Where's web_contents_delegate_android_java? Oh, pi
David Trainor- moved to gerrit 2012/09/20 06:39:26 I think I'm an ant build change CL behind or two.
Ted C 2012/09/20 17:07:04 Yeah, the exported dependent settings for the inpu
David Trainor- moved to gerrit 2012/09/20 21:49:50 Done.
+ '../net/net.gyp:net_java',
+ 'chrome.gyp:chrome_java',
'chrome_android_paks',
'libchromiumtestshell',
],
+ 'variables': {
+ 'package_name': 'chromium_testshell',
+ 'apk_name': 'ChromiumTestShell',
+ 'java_in_dir': 'android/testshell/java',
+ 'resource_dir': '../res',
+ 'native_libs_paths': [ '<(PRODUCT_DIR)/chromium_testshell/libs/<(android_app_abi)/libchromiumtestshell.so', ],
+ 'additional_input_paths': [ '<@(chrome_android_pak_output_resources)', ],
+ },
'actions': [
{
'action_name': 'copy_and_strip_so',
@@ -55,40 +68,8 @@
'<@(_outputs)',
],
},
- {
- 'action_name': 'chromium_testshell_generate_apk',
- 'inputs': [
- '<@(chrome_android_pak_output_resources)',
- '<!@(find android/testshell/java/src -name "*.java")',
- # TODO(dtrainor): Uncomment these once resources are added.
- #'<!@(find android/testshell/java/res -name "*.png")',
- #'<!@(find android/testshell/java/res -name "*.xml")',
- '<(PRODUCT_DIR)/lib.java/chromium_base.jar',
- '<(PRODUCT_DIR)/lib.java/chromium_net.jar',
- '<(PRODUCT_DIR)/lib.java/chromium_media.jar',
- '<(PRODUCT_DIR)/lib.java/chromium_content.jar',
- '<(PRODUCT_DIR)/lib.java/chromium_chrome.jar',
- '<(PRODUCT_DIR)/lib.java/chromium_web_contents_delegate_android.jar',
- '<(PRODUCT_DIR)/chromium_testshell/libs/<(android_app_abi)/libchromiumtestshell.so'
- ],
- 'outputs': [
- '<(PRODUCT_DIR)/chromium_testshell/ChromiumTestShell-debug.apk',
- ],
- 'action': [
- 'ant',
- '-DPRODUCT_DIR=<(ant_build_out)',
- '-DAPP_ABI=<(android_app_abi)',
- '-DANDROID_SDK=<(android_sdk)',
- '-DANDROID_SDK_ROOT=<(android_sdk_root)',
- '-DANDROID_SDK_TOOLS=<(android_sdk_tools)',
- '-DANDROID_SDK_VERSION=<(android_sdk_version)',
- '-DANDROID_GDBSERVER=<(android_gdbserver)',
- '-DCONFIGURATION_NAME=<(CONFIGURATION_NAME)',
- '-buildfile',
- '<(DEPTH)/chrome/android/testshell/java/chromium_testshell_apk.xml',
- ],
- },
],
+ 'includes': [ '../build/java_apk.gypi', ],
},
{
'target_name': 'chrome_android_core',

Powered by Google App Engine
This is Rietveld 408576698