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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/signin/SigninHelper.java

Issue 2784353002: Android: Remove GetApplicationContext part 2 (Closed)
Patch Set: Fix tests Created 3 years, 8 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/android/java/src/org/chromium/chrome/browser/signin/SigninHelper.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninHelper.java
index 594b8f45f30dbcba005de465206bcadff0c6e194..86334e5a846e0bbf960db10dcf454a4218abbe8f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/signin/SigninHelper.java
@@ -121,9 +121,9 @@ public class SigninHelper {
mContext = context;
mProfileSyncService = ProfileSyncService.get();
mSigninManager = SigninManager.get(mContext);
- mAccountTrackerService = AccountTrackerService.get(mContext);
+ mAccountTrackerService = AccountTrackerService.get();
mOAuth2TokenService = OAuth2TokenService.getForProfile(Profile.getLastUsedProfile());
- mChromeSigninController = ChromeSigninController.get(mContext);
+ mChromeSigninController = ChromeSigninController.get();
}
public void validateAccountSettings(boolean accountsChanged) {
@@ -144,7 +144,7 @@ public class SigninHelper {
if (lastSyncAccountName != null && !lastSyncAccountName.isEmpty()) return;
if (!chromePreferenceManager.getSigninPromoShown()
- && AccountManagerHelper.get(mContext).getGoogleAccountNames().size() > 0) {
+ && AccountManagerHelper.get().getGoogleAccountNames().size() > 0) {
chromePreferenceManager.setShowSigninPromo(true);
}
@@ -153,8 +153,8 @@ public class SigninHelper {
String renamedAccount = getNewSignedInAccountName(mContext);
if (accountsChanged && renamedAccount != null) {
- handleAccountRename(ChromeSigninController.get(mContext).getSignedInAccountName(),
- renamedAccount);
+ handleAccountRename(
+ ChromeSigninController.get().getSignedInAccountName(), renamedAccount);
return;
}
@@ -192,7 +192,7 @@ public class SigninHelper {
if (accountsChanged) {
// Account details have changed so inform the token service that credentials
// should now be available.
- mOAuth2TokenService.validateAccounts(mContext, false);
+ mOAuth2TokenService.validateAccounts(false);
}
if (mProfileSyncService != null && AndroidSyncSettings.isSyncEnabled(mContext)) {
@@ -255,7 +255,7 @@ public class SigninHelper {
}
private static boolean accountExists(Context context, Account account) {
- Account[] accounts = AccountManagerHelper.get(context).getGoogleAccounts();
+ Account[] accounts = AccountManagerHelper.get().getGoogleAccounts();
for (Account a : accounts) {
if (a.equals(account)) {
return true;
@@ -299,7 +299,7 @@ public class SigninHelper {
ACCOUNT_RENAMED_PREFS_KEY, null);
// If there is no pending rename, take the name known to ChromeSigninController.
- return name == null ? ChromeSigninController.get(context).getSignedInAccountName() : name;
+ return name == null ? ChromeSigninController.get().getSignedInAccountName() : name;
}
public static void updateAccountRenameData(Context context) {

Powered by Google App Engine
This is Rietveld 408576698