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

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

Issue 1312153003: jni_generator: Pass object parameters as JavaParamRef. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 3 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
Index: base/android/jni_generator/testMultipleJNIAdditionalImport.golden
diff --git a/base/android/jni_generator/testMultipleJNIAdditionalImport.golden b/base/android/jni_generator/testMultipleJNIAdditionalImport.golden
index eb15190a211793ea436c1275bf9a83083e8e4dfa..6de8c219f551e4add8f8d75b9abdb094682235a2 100644
--- a/base/android/jni_generator/testMultipleJNIAdditionalImport.golden
+++ b/base/android/jni_generator/testMultipleJNIAdditionalImport.golden
@@ -27,15 +27,17 @@ jclass g_Foo_clazz = NULL;
// Step 2: method stubs.
-static void DoSomething(JNIEnv* env, jclass jcaller,
- jobject callback1,
- jobject callback2);
+static void DoSomething(JNIEnv* env, const JavaParamRef<jclass>& jcaller,
+ const JavaParamRef<jobject>& callback1,
+ const JavaParamRef<jobject>& callback2);
static void Java_org_chromium_foo_Foo_nativeDoSomething(JNIEnv* env, jclass
jcaller,
jobject callback1,
jobject callback2) {
- return DoSomething(env, jcaller, callback1, callback2);
+ return DoSomething(env, JavaParamRef<jclass>(env, jcaller),
+ JavaParamRef<jobject>(env, callback1), JavaParamRef<jobject>(env,
+ callback2));
}
static base::subtle::AtomicWord g_Foo_calledByNative = 0;
« no previous file with comments | « base/android/jni_generator/testJarJarRemapping.golden ('k') | base/android/jni_generator/testNativeExportsOption.golden » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698