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

Unified Diff: chrome/browser/policy/cloud/user_policy_signin_service.cc

Issue 33173005: Correctly deal with tokens for supervised users in ProfileOAuth2TokenService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 2 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/policy/cloud/user_policy_signin_service.cc
diff --git a/chrome/browser/policy/cloud/user_policy_signin_service.cc b/chrome/browser/policy/cloud/user_policy_signin_service.cc
index db1349a20a46a9a19bff7bec477954537f7dcb22..19e7006f47a510cd2a7de4a398353d55819aac7e 100644
--- a/chrome/browser/policy/cloud/user_policy_signin_service.cc
+++ b/chrome/browser/policy/cloud/user_policy_signin_service.cc
@@ -112,8 +112,11 @@ void UserPolicySigninService::OnRefreshTokenAvailable(
}
std::string username = GetSigninManager()->GetAuthenticatedUsername();
- // Should not have OAuth tokens if the user isn't signed in.
- DCHECK(!username.empty());
+
+ // Ignore OAuth tokens for any account but the primary one.
Bernhard Bauer 2013/10/30 16:23:23 Drew: I changed this. We should here also ignore t
+ if (username != account_id)
+ return;
+
// ProfileOAuth2TokenService now has a refresh token so initialize the
// UserCloudPolicyManager.
InitializeForSignedInUser(username);
« no previous file with comments | « chrome/browser/managed_mode/managed_user_service.cc ('k') | chrome/browser/signin/profile_oauth2_token_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698