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

Unified Diff: sync/engine/sync_scheduler_unittest.cc

Issue 792343004: Standardize usage of virtual/override/final specifiers in sync/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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 | « sync/engine/model_type_sync_worker_impl_unittest.cc ('k') | sync/engine/syncer_proto_util_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/engine/sync_scheduler_unittest.cc
diff --git a/sync/engine/sync_scheduler_unittest.cc b/sync/engine/sync_scheduler_unittest.cc
index 3c86748264454b250ed51669a66e1255e2ebb9df..21d7f28513c756b2d8fca02ef124fa8cc867d89d 100644
--- a/sync/engine/sync_scheduler_unittest.cc
+++ b/sync/engine/sync_scheduler_unittest.cc
@@ -120,7 +120,7 @@ class SyncSchedulerTest : public testing::Test {
MOCK_METHOD1(GetDelay, TimeDelta(const TimeDelta&));
};
- virtual void SetUp() {
+ void SetUp() override {
dir_maker_.SetUp();
syncer_ = new testing::StrictMock<MockSyncer>();
delay_ = NULL;
@@ -172,7 +172,7 @@ class SyncSchedulerTest : public testing::Test {
return TestTimeouts::action_timeout();
}
- virtual void TearDown() {
+ void TearDown() override {
PumpLoop();
scheduler_.reset();
PumpLoop();
@@ -895,14 +895,14 @@ TEST_F(SyncSchedulerTest, ConfigurationMode) {
}
class BackoffTriggersSyncSchedulerTest : public SyncSchedulerTest {
- virtual void SetUp() {
+ void SetUp() override {
SyncSchedulerTest::SetUp();
UseMockDelayProvider();
EXPECT_CALL(*delay(), GetDelay(_))
.WillRepeatedly(Return(TimeDelta::FromMilliseconds(10)));
}
- virtual void TearDown() {
+ void TearDown() override {
StopSyncScheduler();
SyncSchedulerTest::TearDown();
}
« no previous file with comments | « sync/engine/model_type_sync_worker_impl_unittest.cc ('k') | sync/engine/syncer_proto_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698