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

Side by Side Diff: base/android/jni_generator/testMultipleJNIAdditionalImport.golden

Issue 2219923002: JNI: allow either JavaRef or bare objects in Java calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove unreachable line Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // This file is autogenerated by 5 // This file is autogenerated by
6 // base/android/jni_generator/jni_generator.py 6 // base/android/jni_generator/jni_generator.py
7 // For 7 // For
8 // org/chromium/foo/Foo 8 // org/chromium/foo/Foo
9 9
10 #ifndef org_chromium_foo_Foo_JNI 10 #ifndef org_chromium_foo_Foo_JNI
(...skipping 24 matching lines...) Expand all
35 extern "C" __attribute__((visibility("default"))) 35 extern "C" __attribute__((visibility("default")))
36 void Java_org_chromium_foo_Foo_nativeDoSomething(JNIEnv* env, jclass jcaller, 36 void Java_org_chromium_foo_Foo_nativeDoSomething(JNIEnv* env, jclass jcaller,
37 jobject callback1, 37 jobject callback1,
38 jobject callback2) { 38 jobject callback2) {
39 return DoSomething(env, base::android::JavaParamRef<jclass>(env, jcaller), 39 return DoSomething(env, base::android::JavaParamRef<jclass>(env, jcaller),
40 base::android::JavaParamRef<jobject>(env, callback1), 40 base::android::JavaParamRef<jobject>(env, callback1),
41 base::android::JavaParamRef<jobject>(env, callback2)); 41 base::android::JavaParamRef<jobject>(env, callback2));
42 } 42 }
43 43
44 static base::subtle::AtomicWord g_Foo_calledByNative = 0; 44 static base::subtle::AtomicWord g_Foo_calledByNative = 0;
45 static void Java_Foo_calledByNative(JNIEnv* env, jobject callback1, 45 static void Java_Foo_calledByNative(JNIEnv* env, const
46 jobject callback2) { 46 base::android::JavaRefOrBare<jobject>& callback1,
47 const base::android::JavaRefOrBare<jobject>& callback2) {
47 CHECK_CLAZZ(env, Foo_clazz(env), 48 CHECK_CLAZZ(env, Foo_clazz(env),
48 Foo_clazz(env)); 49 Foo_clazz(env));
49 jmethodID method_id = 50 jmethodID method_id =
50 base::android::MethodID::LazyGet< 51 base::android::MethodID::LazyGet<
51 base::android::MethodID::TYPE_STATIC>( 52 base::android::MethodID::TYPE_STATIC>(
52 env, Foo_clazz(env), 53 env, Foo_clazz(env),
53 "calledByNative", 54 "calledByNative",
54 55
55 "(" 56 "("
56 "Lorg/chromium/foo/Bar1$Callback;" 57 "Lorg/chromium/foo/Bar1$Callback;"
57 "Lorg/chromium/foo/Bar2$Callback;" 58 "Lorg/chromium/foo/Bar2$Callback;"
58 ")" 59 ")"
59 "V", 60 "V",
60 &g_Foo_calledByNative); 61 &g_Foo_calledByNative);
61 62
62 env->CallStaticVoidMethod(Foo_clazz(env), 63 env->CallStaticVoidMethod(Foo_clazz(env),
63 method_id, callback1, callback2); 64 method_id, callback1.obj(), callback2.obj());
64 jni_generator::CheckException(env); 65 jni_generator::CheckException(env);
65 66
66 } 67 }
67 68
68 // Step 3: RegisterNatives. 69 // Step 3: RegisterNatives.
69 70
70 static const JNINativeMethod kMethodsFoo[] = { 71 static const JNINativeMethod kMethodsFoo[] = {
71 { "nativeDoSomething", 72 { "nativeDoSomething",
72 "(" 73 "("
73 "Lorg/chromium/foo/Bar1$Callback;" 74 "Lorg/chromium/foo/Bar1$Callback;"
(...skipping 12 matching lines...) Expand all
86 kMethodsFooSize) < 0) { 87 kMethodsFooSize) < 0) {
87 jni_generator::HandleRegistrationError( 88 jni_generator::HandleRegistrationError(
88 env, Foo_clazz(env), __FILE__); 89 env, Foo_clazz(env), __FILE__);
89 return false; 90 return false;
90 } 91 }
91 92
92 return true; 93 return true;
93 } 94 }
94 95
95 #endif // org_chromium_foo_Foo_JNI 96 #endif // org_chromium_foo_Foo_JNI
OLDNEW
« no previous file with comments | « base/android/jni_generator/testFromJavaPGenerics.golden ('k') | base/android/jni_generator/testNativeExportsOnlyOption.golden » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698