diff options
author | torne <torne@chromium.org> | 2015-01-08 11:48:31 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-01-08 19:49:39 +0000 |
commit | 459b8ced84d0ca58df238c9cc50c04d4fc71cbad (patch) | |
tree | d16c58608e43d9049d1da580e9c5abb2f182c3a4 /content/browser/accessibility/browser_accessibility_win.cc | |
parent | 50412b0e582e5210d31ea983e6199178dba1ce1f (diff) | |
download | chromium_src-459b8ced84d0ca58df238c9cc50c04d4fc71cbad.zip chromium_src-459b8ced84d0ca58df238c9cc50c04d4fc71cbad.tar.gz chromium_src-459b8ced84d0ca58df238c9cc50c04d4fc71cbad.tar.bz2 |
android: Improve native JNI export implementation.
To pave the way for making it possible to support both manual JNI
function registration and native JNI exports in the same build, improve
the implementation of native JNI exports by:
1) using a symbol alias instead of a wrapper function to give the
standard Java exported name to non-method native functions. The
previous implementation depended on the function in the .cc file
being inlined into the wrapper function to avoid an extra
indirection, which will not be possible if both modes are supported
at once since there will be a reference to the real function in the
manual registration table and thus it must exist.
2) marking the global used for the Java class reference as potentially
unused, as it's not guaranteed to be referenced if there are no
@CalledByNative Java functions and native exports are being used.
This wasn't causing a problem before as the downstream WebView build
doesn't have unused variable warnings enabled, but was preventing
upstream usage of native export mode.
No functional changes are intended.
BUG=442327
Review URL: https://codereview.chromium.org/841063003
Cr-Commit-Position: refs/heads/master@{#310568}
Diffstat (limited to 'content/browser/accessibility/browser_accessibility_win.cc')
0 files changed, 0 insertions, 0 deletions