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

Unified Diff: chrome/browser/android/signin/signin_manager_android.cc

Issue 2237943002: Remove now-unnecessary .obj() in Java method calls. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@switch-context
Patch Set: Rebase *again* :( 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/signin/signin_manager_android.cc
diff --git a/chrome/browser/android/signin/signin_manager_android.cc b/chrome/browser/android/signin/signin_manager_android.cc
index bfeaaaa81f5312fde66eec680874a2c6034983f2..757cbf78ace0c68b160d00e15375b430202582bd 100644
--- a/chrome/browser/android/signin/signin_manager_android.cc
+++ b/chrome/browser/android/signin/signin_manager_android.cc
@@ -141,8 +141,7 @@ void SigninManagerAndroid::FetchPolicyBeforeSignIn(
// This shouldn't be called when ShouldLoadPolicyForUser() is false, or when
// CheckPolicyBeforeSignIn() failed.
NOTREACHED();
- Java_SigninManager_onPolicyFetchedBeforeSignIn(env,
- java_signin_manager_.obj());
+ Java_SigninManager_onPolicyFetchedBeforeSignIn(env, java_signin_manager_);
}
void SigninManagerAndroid::AbortSignIn(
@@ -217,15 +216,13 @@ void SigninManagerAndroid::OnPolicyRegisterDone(
username_.clear();
}
- Java_SigninManager_onPolicyCheckedBeforeSignIn(env,
- java_signin_manager_.obj(),
- domain.obj());
+ Java_SigninManager_onPolicyCheckedBeforeSignIn(env, java_signin_manager_,
+ domain);
}
void SigninManagerAndroid::OnPolicyFetchDone(bool success) {
Java_SigninManager_onPolicyFetchedBeforeSignIn(
- base::android::AttachCurrentThread(),
- java_signin_manager_.obj());
+ base::android::AttachCurrentThread(), java_signin_manager_);
}
void SigninManagerAndroid::OnBrowsingDataRemoverDone(
@@ -238,8 +235,7 @@ void SigninManagerAndroid::OnBrowsingDataRemoverDone(
ClearLastSignedInUser();
Java_SigninManager_onProfileDataWiped(base::android::AttachCurrentThread(),
- java_signin_manager_.obj(),
- callback.obj());
+ java_signin_manager_, callback);
}
void SigninManagerAndroid::ClearLastSignedInUser(
@@ -290,12 +286,12 @@ void SigninManagerAndroid::GoogleSignedOut(const std::string& account_id,
const std::string& username) {
DCHECK(thread_checker_.CalledOnValidThread());
Java_SigninManager_onNativeSignOut(base::android::AttachCurrentThread(),
- java_signin_manager_.obj());
+ java_signin_manager_);
}
void SigninManagerAndroid::OnSigninAllowedPrefChanged() {
Java_SigninManager_onSigninAllowedByPolicyChanged(
- base::android::AttachCurrentThread(), java_signin_manager_.obj(),
+ base::android::AttachCurrentThread(), java_signin_manager_,
SigninManagerFactory::GetForProfile(profile_)->IsSigninAllowed());
}

Powered by Google App Engine
This is Rietveld 408576698