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

Unified Diff: components/gcm_driver/gcm_driver_desktop.cc

Issue 631343002: [GCM] Fetching OAuth2 tokens periodically in account tracker (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mapper-in-driver
Patch Set: Cleaning up last CR comments Created 6 years, 1 month 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
« no previous file with comments | « components/gcm_driver/gcm_client_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/gcm_driver/gcm_driver_desktop.cc
diff --git a/components/gcm_driver/gcm_driver_desktop.cc b/components/gcm_driver/gcm_driver_desktop.cc
index 6902efbc85e0a91ed49755198fd22a9ac052e9d8..352007a58fa94a698312252773a3e77b490cefdd 100644
--- a/components/gcm_driver/gcm_driver_desktop.cc
+++ b/components/gcm_driver/gcm_driver_desktop.cc
@@ -363,6 +363,10 @@ GCMDriverDesktop::GCMDriverDesktop(
gcm_enabled_(true),
connected_(false),
account_mapper_(new GCMAccountMapper(this)),
+ // Setting to max, to make sure it does not prompt for token reporting
+ // Before reading a reasonable value from the DB, which might be never,
+ // in which case the fetching will be triggered.
+ last_token_fetch_time_(base::Time::Max()),
ui_thread_(ui_thread),
io_thread_(io_thread),
weak_ptr_factory_(this) {
@@ -772,9 +776,10 @@ void GCMDriverDesktop::GCMClientReady(
const base::Time& last_token_fetch_time) {
DCHECK(ui_thread_->RunsTasksOnCurrentThread());
+ last_token_fetch_time_ = last_token_fetch_time;
+
GCMDriver::AddAppHandler(kGCMAccountMapperAppId, account_mapper_.get());
account_mapper_->Initialize(account_mappings);
- last_token_fetch_time_ = last_token_fetch_time;
delayed_task_controller_->SetReady();
}
« no previous file with comments | « components/gcm_driver/gcm_client_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698