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

Unified Diff: base/android/jni_generator/testSingleJNIAdditionalImport.golden

Issue 1279163006: jni_generator: Wrap all native methods in stubs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove separate forward declaration section entirely Created 5 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 | « base/android/jni_generator/testPureNativeMethodsOption.golden ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/android/jni_generator/testSingleJNIAdditionalImport.golden
diff --git a/base/android/jni_generator/testSingleJNIAdditionalImport.golden b/base/android/jni_generator/testSingleJNIAdditionalImport.golden
index 1cf655420531d099c2e651909536eb10f2ecd650..353613946b0756cbcabc394edbb1d1cbcf9288e0 100644
--- a/base/android/jni_generator/testSingleJNIAdditionalImport.golden
+++ b/base/android/jni_generator/testSingleJNIAdditionalImport.golden
@@ -25,10 +25,16 @@ jclass g_Foo_clazz = NULL;
} // namespace
+// Step 2: method stubs.
+
static void DoSomething(JNIEnv* env, jclass jcaller,
jobject callback);
-// Step 2: method stubs.
+static void Java_org_chromium_foo_Foo_nativeDoSomething(JNIEnv* env, jclass
+ jcaller,
+ jobject callback) {
+ return DoSomething(env, jcaller, callback);
+}
static base::subtle::AtomicWord g_Foo_calledByNative = 0;
static void Java_Foo_calledByNative(JNIEnv* env, jobject callback) {
@@ -60,7 +66,7 @@ static const JNINativeMethod kMethodsFoo[] = {
"("
"Lorg/chromium/foo/Bar$Callback;"
")"
-"V", reinterpret_cast<void*>(DoSomething) },
+"V", reinterpret_cast<void*>(Java_org_chromium_foo_Foo_nativeDoSomething) },
};
static bool RegisterNativesImpl(JNIEnv* env) {
« no previous file with comments | « base/android/jni_generator/testPureNativeMethodsOption.golden ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698