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

Unified Diff: chrome/browser/signin/signin_manager.cc

Issue 11649055: OAuth2 sign-in flow for ChromeOS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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/signin/signin_manager.cc
diff --git a/chrome/browser/signin/signin_manager.cc b/chrome/browser/signin/signin_manager.cc
index 24fbd35446fa83ad1d51b5c0953c87202dee94f6..626862faa271444c10bee22efcdf9708e3f04199 100644
--- a/chrome/browser/signin/signin_manager.cc
+++ b/chrome/browser/signin/signin_manager.cc
@@ -130,9 +130,15 @@ void SigninManager::Initialize(Profile* profile) {
TokenService* token_service = TokenServiceFactory::GetForProfile(profile_);
if (token_service) {
token_service->Initialize(GaiaConstants::kChromeSource, profile_);
+ // ChromeOS will kick off TokenService::LoadTokensFromDB from
+ // OAuthLoginManager once the rest of the Profile is fully initialized.
+ // Starting it from here would cause OAuthLoginManager mismatch the origin
+ // of OAuth2 tokens.
+#if !defined(OS_CHROMEOS)
if (!authenticated_username_.empty()) {
token_service->LoadTokensFromDB();
}
+#endif
}
if (!user.empty() && !IsAllowedUsername(user)) {
// User is signed in, but the username is invalid - the administrator must

Powered by Google App Engine
This is Rietveld 408576698