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

Unified Diff: chrome/browser/sync_file_system/sync_process_runner.cc

Issue 666143002: Standardize usage of virtual/override/final in chrome/browser/sync_file_system/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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: chrome/browser/sync_file_system/sync_process_runner.cc
diff --git a/chrome/browser/sync_file_system/sync_process_runner.cc b/chrome/browser/sync_file_system/sync_process_runner.cc
index fa1f100038b656e6af3b89668a3a43f4133d1974..748d21a6d02a260760d0dd474eee96888c6ecf52 100644
--- a/chrome/browser/sync_file_system/sync_process_runner.cc
+++ b/chrome/browser/sync_file_system/sync_process_runner.cc
@@ -26,21 +26,17 @@ class BaseTimerHelper : public SyncProcessRunner::TimerHelper {
public:
BaseTimerHelper() {}
- virtual bool IsRunning() override {
- return timer_.IsRunning();
- }
+ bool IsRunning() override { return timer_.IsRunning(); }
- virtual void Start(const tracked_objects::Location& from_here,
- const base::TimeDelta& delay,
- const base::Closure& closure) override {
+ void Start(const tracked_objects::Location& from_here,
+ const base::TimeDelta& delay,
+ const base::Closure& closure) override {
timer_.Start(from_here, delay, closure);
}
- virtual base::TimeTicks Now() const override {
- return base::TimeTicks::Now();
- }
+ base::TimeTicks Now() const override { return base::TimeTicks::Now(); }
- virtual ~BaseTimerHelper() {}
+ ~BaseTimerHelper() override {}
private:
base::OneShotTimer<SyncProcessRunner> timer_;

Powered by Google App Engine
This is Rietveld 408576698