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

Unified Diff: testing/android/native_test.gyp

Issue 10798010: Uses gyp "rules" rather than "actions" templates for the JNI generator. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Typo on content_jni.gypi Created 8 years, 5 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 | « net/proxy/proxy_config_service_android.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/android/native_test.gyp
diff --git a/testing/android/native_test.gyp b/testing/android/native_test.gyp
index 90d080b48d8ac422790a5080cf373f8e54a83b35..fb21f27786d71063f0ae8d4c7110548465d0dc73 100644
--- a/testing/android/native_test.gyp
+++ b/testing/android/native_test.gyp
@@ -59,7 +59,7 @@
'-Wl,--exclude-libs=ALL',
],
},
- 'dependencies': [
+ 'dependencies': [
'../../base/base.gyp:base',
'../../base/base.gyp:test_support_base',
'../gtest.gyp:gtest',
@@ -69,26 +69,13 @@
{
'target_name': 'native_test_jni_headers',
'type': 'none',
- 'actions': [
- {
- 'action_name': 'generate_jni_headers',
- 'inputs': [
- '../../base/android/jni_generator/jni_generator.py',
- 'java/src/org/chromium/native_test/ChromeNativeTestActivity.java'
- ],
- 'outputs': [
- '<(SHARED_INTERMEDIATE_DIR)/testing/android/jni/'
- 'chrome_native_test_activity_jni.h',
- ],
- 'action': [
- 'python',
- '../../base/android/jni_generator/jni_generator.py',
- '-o',
- '<@(_inputs)',
- '<@(_outputs)',
- ],
- }
+ 'sources': [
+ 'java/src/org/chromium/native_test/ChromeNativeTestActivity.java'
],
+ 'variables': {
+ 'jni_gen_dir': 'testing',
+ },
+ 'includes': [ '../../build/jni_generator.gypi' ],
# So generated jni headers can be found by targets that
# depend on this.
'direct_dependent_settings': {
« no previous file with comments | « net/proxy/proxy_config_service_android.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698