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

Unified Diff: components/signin/core/browser/gaia_cookie_manager_service.cc

Issue 2697563002: Pass time since profile was loaded and request count on each ListAccount request to Gaia. (Closed)
Patch Set: Nit Created 3 years, 10 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/gaia_cookie_manager_service.cc
diff --git a/components/signin/core/browser/gaia_cookie_manager_service.cc b/components/signin/core/browser/gaia_cookie_manager_service.cc
index 28f30f1d2ffab1aa02322776bfcc0253688d930f..2b2e414f7a795aad657837fcce09d58c6ec05507 100644
--- a/components/signin/core/browser/gaia_cookie_manager_service.cc
+++ b/components/signin/core/browser/gaia_cookie_manager_service.cc
@@ -298,7 +298,11 @@ GaiaCookieManagerService::GaiaCookieManagerService(
fetcher_retries_(0),
source_(source),
external_cc_result_fetched_(false),
- list_accounts_stale_(true) {
+ list_accounts_stale_(true),
+ // |GaiaCookieManagerService| is created as soon as the profle is
+ // initialized so it is acceptable to use of this
+ // |GaiaCookieManagerService| as the time when the profile is loaded.
+ profile_load_time_(base::Time::Now()) {
DCHECK(!source_.empty());
}
@@ -757,7 +761,7 @@ void GaiaCookieManagerService::StartFetchingListAccounts() {
gaia_auth_fetcher_.reset(signin_client_->CreateGaiaAuthFetcher(
this, GetSourceForRequest(requests_.front()),
signin_client_->GetURLRequestContext()));
- gaia_auth_fetcher_->StartListAccounts();
+ gaia_auth_fetcher_->StartListAccounts(profile_load_time_);
}
void GaiaCookieManagerService::HandleNextRequest() {

Powered by Google App Engine
This is Rietveld 408576698