jni_generator: Wrap all native methods in stubs.
commitb37ad14e10041f15a4467022d5169be4dac73ceb
authorTorne (Richard Coles) <torne@chromium.org>
Fri, 14 Aug 2015 12:41:14 +0000 (14 13:41 +0100)
committerTorne (Richard Coles) <torne@chromium.org>
Fri, 14 Aug 2015 12:42:22 +0000 (14 12:42 +0000)
tree30a15f5612c3d85cb33af25d61b113505825c374
parent109d90ca91cf28023f47478c6685a4f9c31b789e
jni_generator: Wrap all native methods in stubs.

Instead of only wrapping native methods which call C++ methods, wrap
them all. This doesn't require any changes to the implementations, but
makes the handling of the two kinds of method more consistent in
preparation for introducing return type/parameter wrapping in future
changes.

This also changes the way that some of the stub code is generated to
make it less different depending whether native exports are in use, and
moves "extern "C"" declarations to be on only the specifically exported
symbols instead of applying to all forward declarations.

BUG=519562
R=rmcilroy@chromium.org

Review URL: https://codereview.chromium.org/1279163006 .

Cr-Commit-Position: refs/heads/master@{#343370}
15 files changed:
base/android/jni_generator/golden_sample_for_tests_jni.h
base/android/jni_generator/jni_generator.py
base/android/jni_generator/testEagerCalledByNativesOption.golden
base/android/jni_generator/testInnerClassNatives.golden
base/android/jni_generator/testInnerClassNativesBothInnerAndOuter.golden
base/android/jni_generator/testInnerClassNativesMultiple.golden
base/android/jni_generator/testJNIInitNativeNameOption.golden
base/android/jni_generator/testJarJarRemapping.golden
base/android/jni_generator/testMultipleJNIAdditionalImport.golden
base/android/jni_generator/testNativeExportsOption.golden
base/android/jni_generator/testNativeExportsOptionalOption.golden
base/android/jni_generator/testNatives.golden
base/android/jni_generator/testNativesLong.golden
base/android/jni_generator/testPureNativeMethodsOption.golden
base/android/jni_generator/testSingleJNIAdditionalImport.golden