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

Unified Diff: build/jni_generator.gypi

Issue 454923002: Don't register JNI methods for the android_webview. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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 | « build/jar_file_jni_generator.gypi ('k') | build/linker_script_copy.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/jni_generator.gypi
diff --git a/build/jni_generator.gypi b/build/jni_generator.gypi
index da99331ec522bfc27a3e544c40614bdd025c76ea..48ec24eee20a2df4d76606e0edb55e782a741d6a 100644
--- a/build/jni_generator.gypi
+++ b/build/jni_generator.gypi
@@ -36,7 +36,11 @@
'jni_generator_includes%': (
'base/android/jni_generator/jni_generator_helper.h'
),
+ 'native_exports%': '',
},
+ 'dependencies': [
+ '<(DEPTH)/build/linker_script_copy.gyp:linker_script_copy',
+ ],
'rules': [
{
'rule_name': 'generate_jni_headers',
@@ -61,6 +65,7 @@
'<(jni_generator_jarjar_file)',
'--ptr_type',
'<(jni_generator_ptr_type)',
+ '<(native_exports)',
],
'message': 'Generating JNI bindings from <(RULE_INPUT_PATH)',
'process_outputs_as_sources': 1,
@@ -81,4 +86,28 @@
# This target exports a hard dependency because it generates header
# files.
'hard_dependency': 1,
+ 'conditions': [
+ ['android_webview_build==1', {
+ 'variables': {
+ 'native_exports%': '--native_exports',
+ },
+ 'dependencies': [
+ '<(DEPTH)/build/linker_script_copy.gyp:linker_script_copy',
+ ],
+ 'conditions': [
+ ['component=="static_library"', {
+ 'link_settings': {
+ 'ldflags': [
+ # Only export symbols that are specified in version script.
+ '-Wl,--version-script=<(android_linker_script)',
+ ],
+ 'ldflags!': [
+ '-Wl,--exclude-libs=ALL',
+ ],
+ },
+ }],
+ ],
+ }],
+ ],
}
+
« no previous file with comments | « build/jar_file_jni_generator.gypi ('k') | build/linker_script_copy.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698