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

Unified Diff: android/jni_generator/golden_sample_for_tests_jni.h

Issue 2050803003: Update to Chromium //base at Chromium commit e3a753f17bac62738b0dbf0b36510f767b081e4b. (Closed) Base URL: https://github.com/domokit/base.git@master
Patch Set: Created 4 years, 6 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: android/jni_generator/golden_sample_for_tests_jni.h
diff --git a/android/jni_generator/golden_sample_for_tests_jni.h b/android/jni_generator/golden_sample_for_tests_jni.h
index 8979697b8764f0e815343fa3e3940e6c6343d6fa..fd1a632b0020883852c75d672ecd08686e81cc4e 100644
--- a/android/jni_generator/golden_sample_for_tests_jni.h
+++ b/android/jni_generator/golden_sample_for_tests_jni.h
@@ -41,14 +41,16 @@ namespace android {
// Step 2: method stubs.
-static jlong Init(JNIEnv* env, jobject jcaller,
- jstring param);
+static jlong Init(JNIEnv* env,
+ const JavaParamRef<jobject>& jcaller,
+ const JavaParamRef<jstring>& param);
static jlong Java_org_chromium_example_jni_1generator_SampleForTests_nativeInit(
JNIEnv* env,
jobject jcaller,
jstring param) {
- return Init(env, jcaller, param);
+ return Init(env, JavaParamRef<jobject>(env, jcaller),
+ JavaParamRef<jstring>(env, param));
}
static void
@@ -58,46 +60,51 @@ Java_org_chromium_example_jni_1generator_SampleForTests_nativeDestroy(
jlong nativeCPPClass) {
CPPClass* native = reinterpret_cast<CPPClass*>(nativeCPPClass);
CHECK_NATIVE_PTR(env, jcaller, native, "Destroy");
- return native->Destroy(env, jcaller);
+ return native->Destroy(env, JavaParamRef<jobject>(env, jcaller));
}
-static jdouble GetDoubleFunction(JNIEnv* env, jobject jcaller);
+static jdouble GetDoubleFunction(JNIEnv* env,
+ const JavaParamRef<jobject>& jcaller);
static jdouble
Java_org_chromium_example_jni_1generator_SampleForTests_nativeGetDoubleFunction(
JNIEnv* env,
jobject jcaller) {
- return GetDoubleFunction(env, jcaller);
+ return GetDoubleFunction(env, JavaParamRef<jobject>(env, jcaller));
}
-static jfloat GetFloatFunction(JNIEnv* env, jclass jcaller);
+static jfloat GetFloatFunction(JNIEnv* env,
+ const JavaParamRef<jclass>& jcaller);
static jfloat
Java_org_chromium_example_jni_1generator_SampleForTests_nativeGetFloatFunction(
JNIEnv* env,
jclass jcaller) {
- return GetFloatFunction(env, jcaller);
+ return GetFloatFunction(env, JavaParamRef<jclass>(env, jcaller));
}
-static void SetNonPODDatatype(JNIEnv* env, jobject jcaller,
- jobject rect);
+static void SetNonPODDatatype(JNIEnv* env,
+ const JavaParamRef<jobject>& jcaller,
+ const JavaParamRef<jobject>& rect);
static void
Java_org_chromium_example_jni_1generator_SampleForTests_nativeSetNonPODDatatype(
JNIEnv* env,
jobject jcaller,
jobject rect) {
- return SetNonPODDatatype(env, jcaller, rect);
+ return SetNonPODDatatype(env, JavaParamRef<jobject>(env, jcaller),
+ JavaParamRef<jobject>(env, rect));
}
-static ScopedJavaLocalRef<jobject> GetNonPODDatatype(JNIEnv* env,
- jobject jcaller);
+static ScopedJavaLocalRef<jobject> GetNonPODDatatype(
+ JNIEnv* env,
+ const JavaParamRef<jobject>& jcaller);
static jobject
Java_org_chromium_example_jni_1generator_SampleForTests_nativeGetNonPODDatatype(
JNIEnv* env,
jobject jcaller) {
- return GetNonPODDatatype(env, jcaller).Release();
+ return GetNonPODDatatype(env, JavaParamRef<jobject>(env, jcaller)).Release();
}
static jint
@@ -107,7 +114,7 @@ Java_org_chromium_example_jni_1generator_SampleForTests_nativeMethod(
jlong nativeCPPClass) {
CPPClass* native = reinterpret_cast<CPPClass*>(nativeCPPClass);
CHECK_NATIVE_PTR(env, jcaller, native, "Method", 0);
- return native->Method(env, jcaller);
+ return native->Method(env, JavaParamRef<jobject>(env, jcaller));
}
static jdouble
@@ -118,7 +125,7 @@ Java_org_chromium_example_jni_1generator_SampleForTests_nativeMethodOtherP0(
CPPClass::InnerClass* native =
reinterpret_cast<CPPClass::InnerClass*>(nativePtr);
CHECK_NATIVE_PTR(env, jcaller, native, "MethodOtherP0", 0);
- return native->MethodOtherP0(env, jcaller);
+ return native->MethodOtherP0(env, JavaParamRef<jobject>(env, jcaller));
}
static void
@@ -129,7 +136,8 @@ Java_org_chromium_example_jni_1generator_SampleForTests_nativeAddStructB(
jobject b) {
CPPClass* native = reinterpret_cast<CPPClass*>(nativeCPPClass);
CHECK_NATIVE_PTR(env, jcaller, native, "AddStructB");
- return native->AddStructB(env, jcaller, b);
+ return native->AddStructB(env, JavaParamRef<jobject>(env, jcaller),
+ JavaParamRef<jobject>(env, b));
}
static void
@@ -139,7 +147,8 @@ Java_org_chromium_example_jni_1generator_SampleForTests_nativeIterateAndDoSometh
jlong nativeCPPClass) {
CPPClass* native = reinterpret_cast<CPPClass*>(nativeCPPClass);
CHECK_NATIVE_PTR(env, jcaller, native, "IterateAndDoSomethingWithStructB");
- return native->IterateAndDoSomethingWithStructB(env, jcaller);
+ return native->IterateAndDoSomethingWithStructB(
+ env, JavaParamRef<jobject>(env, jcaller));
}
static jstring
@@ -149,7 +158,8 @@ Java_org_chromium_example_jni_1generator_SampleForTests_nativeReturnAString(
jlong nativeCPPClass) {
CPPClass* native = reinterpret_cast<CPPClass*>(nativeCPPClass);
CHECK_NATIVE_PTR(env, jcaller, native, "ReturnAString", NULL);
- return native->ReturnAString(env, jcaller).Release();
+ return native->ReturnAString(env, JavaParamRef<jobject>(env, jcaller))
+ .Release();
}
static base::subtle::AtomicWord g_SampleForTests_javaMethod = 0;
« no previous file with comments | « android/java/src/org/chromium/base/library_loader/ModernLinker.java ('k') | android/jni_generator/jni_generator.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698