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

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: chromeos 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..0aa6f2c05953f14849aaa0a0bcd5fd0a93960bbc 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 if the user isn't signed in.
+ if (username.empty())
+ return;
+
// ProfileOAuth2TokenService now has a refresh token so initialize the
// UserCloudPolicyManager.
InitializeForSignedInUser(username);

Powered by Google App Engine
This is Rietveld 408576698