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

Side by Side Diff: chrome/browser/android/signin/signin_manager_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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #include <vector> 5 #include <vector>
6 6
7 #include "chrome/browser/android/signin/signin_manager_android.h" 7 #include "chrome/browser/android/signin/signin_manager_android.h"
8 8
9 #include "base/android/jni_android.h" 9 #include "base/android/jni_android.h"
10 #include "base/android/jni_array.h" 10 #include "base/android/jni_array.h"
(...skipping 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
277 jboolean SigninManagerAndroid::IsSignedInOnNative(JNIEnv* env, jobject obj) { 277 jboolean SigninManagerAndroid::IsSignedInOnNative(JNIEnv* env, jobject obj) {
278 return SigninManagerFactory::GetForProfile(profile_)->IsAuthenticated(); 278 return SigninManagerFactory::GetForProfile(profile_)->IsAuthenticated();
279 } 279 }
280 280
281 void SigninManagerAndroid::OnSigninAllowedPrefChanged() { 281 void SigninManagerAndroid::OnSigninAllowedPrefChanged() {
282 Java_SigninManager_onSigninAllowedByPolicyChanged( 282 Java_SigninManager_onSigninAllowedByPolicyChanged(
283 base::android::AttachCurrentThread(), java_signin_manager_.obj(), 283 base::android::AttachCurrentThread(), java_signin_manager_.obj(),
284 SigninManagerFactory::GetForProfile(profile_)->IsSigninAllowed()); 284 SigninManagerFactory::GetForProfile(profile_)->IsSigninAllowed());
285 } 285 }
286 286
287 static jlong Init(JNIEnv* env, jobject obj) { 287 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& obj) {
288 SigninManagerAndroid* signin_manager_android = 288 SigninManagerAndroid* signin_manager_android =
289 new SigninManagerAndroid(env, obj); 289 new SigninManagerAndroid(env, obj);
290 return reinterpret_cast<intptr_t>(signin_manager_android); 290 return reinterpret_cast<intptr_t>(signin_manager_android);
291 } 291 }
292 292
293 static jboolean ShouldLoadPolicyForUser(JNIEnv* env, 293 static jboolean ShouldLoadPolicyForUser(
294 jobject obj, 294 JNIEnv* env,
295 jstring j_username) { 295 const JavaParamRef<jobject>& obj,
296 const JavaParamRef<jstring>& j_username) {
296 #if defined(ENABLE_CONFIGURATION_POLICY) 297 #if defined(ENABLE_CONFIGURATION_POLICY)
297 std::string username = 298 std::string username =
298 base::android::ConvertJavaStringToUTF8(env, j_username); 299 base::android::ConvertJavaStringToUTF8(env, j_username);
299 return !policy::BrowserPolicyConnector::IsNonEnterpriseUser(username); 300 return !policy::BrowserPolicyConnector::IsNonEnterpriseUser(username);
300 #else 301 #else
301 return false; 302 return false;
302 #endif 303 #endif
303 } 304 }
304 305
305 // static 306 // static
306 bool SigninManagerAndroid::Register(JNIEnv* env) { 307 bool SigninManagerAndroid::Register(JNIEnv* env) {
307 return RegisterNativesImpl(env); 308 return RegisterNativesImpl(env);
308 } 309 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698