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

Unified Diff: net/cert/x509_util_android.cc

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: net/cert/x509_util_android.cc
diff --git a/net/cert/x509_util_android.cc b/net/cert/x509_util_android.cc
index 9648ffb91f09b39058059eca555d50c7f0298459..2a91f4a2792a580f1b5f5450777f0e10fa7a27e7 100644
--- a/net/cert/x509_util_android.cc
+++ b/net/cert/x509_util_android.cc
@@ -12,12 +12,12 @@
namespace net {
-void NotifyKeyChainChanged(JNIEnv* env, jclass clazz) {
+void NotifyKeyChainChanged(JNIEnv* env, const JavaParamRef<jclass>& clazz) {
CertDatabase::GetInstance()->OnAndroidKeyChainChanged();
}
void RecordCertVerifyCapabilitiesHistogram(JNIEnv* env,
- jclass clazz,
+ const JavaParamRef<jclass>& clazz,
jboolean found_system_trust_roots) {
// Only record the histogram for 4.2 and up. Before 4.2, the platform doesn't
// return the certificate chain anyway.
@@ -27,7 +27,9 @@ void RecordCertVerifyCapabilitiesHistogram(JNIEnv* env,
}
}
-ScopedJavaLocalRef<jobject> GetApplicationContext(JNIEnv* env, jclass clazz) {
+ScopedJavaLocalRef<jobject> GetApplicationContext(
+ JNIEnv* env,
+ const JavaParamRef<jclass>& clazz) {
ScopedJavaLocalRef<jobject> r;
// Must use Reset to force creation of a new local ref, instead of trying to
// adopt the global-ref'ed jobject as a local ref as the constructor would.

Powered by Google App Engine
This is Rietveld 408576698