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

Unified Diff: chrome/browser/sync_file_system/sync_process_runner_unittest.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
« no previous file with comments | « chrome/browser/sync_file_system/sync_process_runner.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync_file_system/sync_process_runner_unittest.cc
diff --git a/chrome/browser/sync_file_system/sync_process_runner_unittest.cc b/chrome/browser/sync_file_system/sync_process_runner_unittest.cc
index 98594ac1e665f9f544fbaf27b9dfbbba62b3697d..a1bd4b7f62288ab8839e172a57a586895256d4ca 100644
--- a/chrome/browser/sync_file_system/sync_process_runner_unittest.cc
+++ b/chrome/browser/sync_file_system/sync_process_runner_unittest.cc
@@ -16,15 +16,11 @@ namespace {
class FakeClient : public SyncProcessRunner::Client {
public:
FakeClient() : service_state_(SYNC_SERVICE_RUNNING) {}
- virtual ~FakeClient() {}
+ ~FakeClient() override {}
- virtual SyncServiceState GetSyncServiceState() override {
- return service_state_;
- }
+ SyncServiceState GetSyncServiceState() override { return service_state_; }
- virtual SyncFileSystemService* GetSyncService() override {
- return nullptr;
- }
+ SyncFileSystemService* GetSyncService() override { return nullptr; }
void set_service_state(SyncServiceState service_state) {
service_state_ = service_state;
@@ -39,22 +35,18 @@ class FakeClient : public SyncProcessRunner::Client {
class FakeTimerHelper : public SyncProcessRunner::TimerHelper {
public:
FakeTimerHelper() {}
- virtual ~FakeTimerHelper() {}
+ ~FakeTimerHelper() override {}
- virtual bool IsRunning() override {
- return !timer_task_.is_null();
- }
+ bool IsRunning() override { return !timer_task_.is_null(); }
- 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 {
scheduled_time_ = current_time_ + delay;
timer_task_ = closure;
}
- virtual base::TimeTicks Now() const override {
- return current_time_;
- }
+ base::TimeTicks Now() const override { return current_time_; }
void SetCurrentTime(const base::TimeTicks& current_time) {
current_time_ = current_time;
@@ -94,13 +86,12 @@ class FakeSyncProcessRunner : public SyncProcessRunner {
max_parallel_task_(max_parallel_task) {
}
- virtual void StartSync(const SyncStatusCallback& callback) override {
+ void StartSync(const SyncStatusCallback& callback) override {
EXPECT_LT(running_tasks_.size(), max_parallel_task_);
running_tasks_.push(callback);
}
- virtual ~FakeSyncProcessRunner() {
- }
+ ~FakeSyncProcessRunner() override {}
void UpdateChanges(int num_changes) {
OnChangesUpdated(num_changes);
« no previous file with comments | « chrome/browser/sync_file_system/sync_process_runner.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698