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

Unified Diff: components/cryptauth/sync_scheduler_impl.cc

Issue 2837863004: Use base::OneShotTimer instead of base::Timer(false, false)
Patch Set: rebase Created 3 years, 8 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
« no previous file with comments | « components/component_updater/timer.cc ('k') | components/domain_reliability/util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cryptauth/sync_scheduler_impl.cc
diff --git a/components/cryptauth/sync_scheduler_impl.cc b/components/cryptauth/sync_scheduler_impl.cc
index d073eddc03bda4353cca2b8640a11bf371f52220..7ed531e75455d61db72d3a79a7922ced96ccdae8 100644
--- a/components/cryptauth/sync_scheduler_impl.cc
+++ b/components/cryptauth/sync_scheduler_impl.cc
@@ -117,9 +117,7 @@ void SyncSchedulerImpl::OnTimerFired() {
}
std::unique_ptr<base::Timer> SyncSchedulerImpl::CreateTimer() {
- bool retain_user_task = false;
- bool is_repeating = false;
- return base::MakeUnique<base::Timer>(retain_user_task, is_repeating);
+ return base::MakeUnique<base::OneShotTimer>();
}
void SyncSchedulerImpl::ScheduleNextSync(const base::TimeDelta& sync_delta) {
« no previous file with comments | « components/component_updater/timer.cc ('k') | components/domain_reliability/util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698