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

Unified Diff: chrome/browser/sync/test/integration/migration_test.cc

Issue 686563003: Standardize usage of virtual/override/final specifiers. (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/test/integration/migration_test.cc
diff --git a/chrome/browser/sync/test/integration/migration_test.cc b/chrome/browser/sync/test/integration/migration_test.cc
index ef2ac1ec43fa88065974b225ec3551b375394f66..753865e8a1f3e37e38f54a74d7ed651957ee7edd 100644
--- a/chrome/browser/sync/test/integration/migration_test.cc
+++ b/chrome/browser/sync/test/integration/migration_test.cc
@@ -70,7 +70,7 @@ MigrationList MakeList(syncer::ModelType type1,
class MigrationTest : public SyncTest {
public:
explicit MigrationTest(TestType test_type) : SyncTest(test_type) {}
- virtual ~MigrationTest() {}
+ ~MigrationTest() override {}
enum TriggerMethod { MODIFY_PREF, MODIFY_BOOKMARK, TRIGGER_NOTIFICATION };
@@ -216,7 +216,7 @@ class MigrationTest : public SyncTest {
class MigrationSingleClientTest : public MigrationTest {
public:
MigrationSingleClientTest() : MigrationTest(SINGLE_CLIENT_LEGACY) {}
- virtual ~MigrationSingleClientTest() {}
+ ~MigrationSingleClientTest() override {}
void RunSingleClientMigrationTest(const MigrationList& migration_list,
TriggerMethod trigger_method) {
@@ -341,7 +341,7 @@ IN_PROC_BROWSER_TEST_F(MigrationSingleClientTest, AllTypesWithNigoriAtOnce) {
class MigrationTwoClientTest : public MigrationTest {
public:
MigrationTwoClientTest() : MigrationTest(TWO_CLIENT_LEGACY) {}
- virtual ~MigrationTwoClientTest() {}
+ ~MigrationTwoClientTest() override {}
// Helper function that verifies that preferences sync still works.
void VerifyPrefSync() {
@@ -426,7 +426,7 @@ class MigrationReconfigureTest : public MigrationTwoClientTest {
// Do not add optional datatypes.
}
- virtual ~MigrationReconfigureTest() {}
+ ~MigrationReconfigureTest() override {}
private:
DISALLOW_COPY_AND_ASSIGN(MigrationReconfigureTest);

Powered by Google App Engine
This is Rietveld 408576698