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

Unified Diff: components/signin/core/browser/profile_oauth2_token_service.h

Issue 2939443004: Load credentials on desktop even when there is no authenticated account ID. (Closed)
Patch Set: Unit test Created 3 years, 6 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: components/signin/core/browser/profile_oauth2_token_service.h
diff --git a/components/signin/core/browser/profile_oauth2_token_service.h b/components/signin/core/browser/profile_oauth2_token_service.h
index fadf1417bd1f7b6334c18bdbb471e65c614f8d39..b7beb4eec4a3c54df6dcf10a934e5c6daa0d00da 100644
--- a/components/signin/core/browser/profile_oauth2_token_service.h
+++ b/components/signin/core/browser/profile_oauth2_token_service.h
@@ -42,12 +42,9 @@ class ProfileOAuth2TokenService : public OAuth2TokenService,
// Loads credentials from a backing persistent store to make them available
// after service is used between profile restarts.
//
- // Only call this method if there is at least one account connected to the
- // profile, otherwise startup will cause unneeded work on the IO thread. The
- // primary account is specified with the |primary_account_id| argument. If
- // empty, no credentials will be loaded. For a regular profile, the primary
- // account id comes from SigninManager. For a supervised user, the id comes
- // from SupervisedUserService.
+ // The primary account is specified with the |primary_account_id| argument.
+ // For a regular profile, the primary account id comes from SigninManager.
+ // For a supervised user, the id comes from SupervisedUserService.
virtual void LoadCredentials(const std::string& primary_account_id);
// Updates a |refresh_token| for an |account_id|. Credentials are persisted,

Powered by Google App Engine
This is Rietveld 408576698