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

Unified Diff: base/android/jni_generator/golden_sample_for_tests_jni.h

Issue 1308363003: Revert of jni_generator: Make all object-returning natives return ScopedJavaLocalRef. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/field_trial_list.cc ('k') | base/android/jni_generator/jni_generator.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/android/jni_generator/golden_sample_for_tests_jni.h
diff --git a/base/android/jni_generator/golden_sample_for_tests_jni.h b/base/android/jni_generator/golden_sample_for_tests_jni.h
index 974cfffde8a17a4ca08a32f5ef5e8b827bad390e..e146acada34001dbb25a27b2e23988f1fbbd8040 100644
--- a/base/android/jni_generator/golden_sample_for_tests_jni.h
+++ b/base/android/jni_generator/golden_sample_for_tests_jni.h
@@ -87,13 +87,12 @@
return SetNonPODDatatype(env, jcaller, rect);
}
-static ScopedJavaLocalRef<jobject> GetNonPODDatatype(JNIEnv* env, jobject
- jcaller);
+static jobject GetNonPODDatatype(JNIEnv* env, jobject jcaller);
static jobject
Java_org_chromium_example_jni_1generator_SampleForTests_nativeGetNonPODDatatype(JNIEnv*
env, jobject jcaller) {
- return GetNonPODDatatype(env, jcaller).Release();
+ return GetNonPODDatatype(env, jcaller);
}
static jint
@@ -244,8 +243,8 @@
}
static base::subtle::AtomicWord g_InnerStructA_create = 0;
-static ScopedJavaLocalRef<jobject> Java_InnerStructA_create(JNIEnv* env, jlong
- l,
+static base::android::ScopedJavaLocalRef<jobject>
+ Java_InnerStructA_create(JNIEnv* env, jlong l,
JniIntWrapper i,
jstring s) {
/* Must call RegisterNativesImpl() */
@@ -269,7 +268,7 @@
env->CallStaticObjectMethod(InnerStructA_clazz(env),
method_id, l, as_jint(i), s);
jni_generator::CheckException(env);
- return ScopedJavaLocalRef<jobject>(env, ret);
+ return base::android::ScopedJavaLocalRef<jobject>(env, ret);
}
static base::subtle::AtomicWord g_SampleForTests_addStructA = 0;
@@ -343,8 +342,8 @@
}
static base::subtle::AtomicWord g_InnerStructB_getValue = 0;
-static ScopedJavaLocalRef<jstring> Java_InnerStructB_getValue(JNIEnv* env,
- jobject obj) {
+static base::android::ScopedJavaLocalRef<jstring>
+ Java_InnerStructB_getValue(JNIEnv* env, jobject obj) {
/* Must call RegisterNativesImpl() */
CHECK_CLAZZ(env, obj,
InnerStructB_clazz(env), NULL);
@@ -363,7 +362,7 @@
static_cast<jstring>(env->CallObjectMethod(obj,
method_id));
jni_generator::CheckException(env);
- return ScopedJavaLocalRef<jstring>(env, ret);
+ return base::android::ScopedJavaLocalRef<jstring>(env, ret);
}
// Step 3: RegisterNatives.
« no previous file with comments | « base/android/field_trial_list.cc ('k') | base/android/jni_generator/jni_generator.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698