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

Unified Diff: components/domain_reliability/util.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/cryptauth/sync_scheduler_impl.cc ('k') | components/drive/file_write_watcher.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/domain_reliability/util.cc
diff --git a/components/domain_reliability/util.cc b/components/domain_reliability/util.cc
index 1b3431a09fecde0aea09986d7e329bb63b88d0b5..a8cfb11e4432a3acab0dd48a5e222f6628adae46 100644
--- a/components/domain_reliability/util.cc
+++ b/components/domain_reliability/util.cc
@@ -205,8 +205,7 @@ namespace {
class ActualTimer : public MockableTime::Timer {
public:
- // Initialize base timer with retain_user_info and is_repeating false.
- ActualTimer() : base_timer_(false, false) {}
+ ActualTimer() {}
~ActualTimer() override {}
@@ -222,7 +221,7 @@ class ActualTimer : public MockableTime::Timer {
bool IsRunning() override { return base_timer_.IsRunning(); }
private:
- base::Timer base_timer_;
+ base::OneShotTimer base_timer_;
};
} // namespace
« no previous file with comments | « components/cryptauth/sync_scheduler_impl.cc ('k') | components/drive/file_write_watcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698