Standardize usage of virtual/override/final in chrome/browser/sync/

chrome/browser/sync/test/integration/sync_errors_test.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/prefs/pref_member.h"
 #include "base/prefs/pref_service.h"
 #include "base/run_loop.h"
 #include "chrome/browser/sync/profile_sync_service.h"
 #include "chrome/browser/sync/test/integration/bookmarks_helper.h"
 #include "chrome/browser/sync/test/integration/passwords_helper.h"
 #include "chrome/browser/sync/test/integration/profile_sync_service_harness.h"
 #include "chrome/browser/sync/test/integration/single_client_status_change_checker.h"
 #include "chrome/browser/sync/test/integration/sync_integration_test_util.h"
 #include "chrome/browser/sync/test/integration/sync_test.h"
 #include "chrome/common/pref_names.h"
 #include "google_apis/gaia/google_service_auth_error.h"
 #include "sync/protocol/sync_protocol_error.h"
 
 using bookmarks_helper::AddFolder;
 using bookmarks_helper::SetTitle;
 using sync_integration_test_util::AwaitCommitActivityCompletion;
 
 namespace {
 
 class SyncDisabledChecker : public SingleClientStatusChangeChecker {
  public:
   explicit SyncDisabledChecker(ProfileSyncService* service)
       : SingleClientStatusChangeChecker(service) {}
 
-  virtual bool IsExitConditionSatisfied() override {
+  bool IsExitConditionSatisfied() override {
     return !service()->setup_in_progress() &&
            !service()->HasSyncSetupCompleted();
   }
 
-  virtual std::string GetDebugMessage() const override {
-    return "Sync Disabled";
-  }
+  std::string GetDebugMessage() const override { return "Sync Disabled"; }
 };
 
 class TypeDisabledChecker : public SingleClientStatusChangeChecker {
  public:
   explicit TypeDisabledChecker(ProfileSyncService* service,
                                syncer::ModelType type)
       : SingleClientStatusChangeChecker(service), type_(type) {}
 
-  virtual bool IsExitConditionSatisfied() override {
+  bool IsExitConditionSatisfied() override {
     return !service()->GetActiveDataTypes().Has(type_);
   }
 
-  virtual std::string GetDebugMessage() const override {
-    return "Type disabled";
-  }
+  std::string GetDebugMessage() const override { return "Type disabled"; }
  private:
    syncer::ModelType type_;
 };
 
 bool AwaitSyncDisabled(ProfileSyncService* service) {
   SyncDisabledChecker checker(service);
   checker.Wait();
   return !checker.TimedOut();
 }
 
@@ skipping 12 matching lines @@
  private:
   DISALLOW_COPY_AND_ASSIGN(SyncErrorTest);
 };
 
 // Helper class that waits until the sync engine has hit an actionable error.
 class ActionableErrorChecker : public SingleClientStatusChangeChecker {
  public:
   explicit ActionableErrorChecker(ProfileSyncService* service)
       : SingleClientStatusChangeChecker(service) {}
 
-  virtual ~ActionableErrorChecker() {}
+  ~ActionableErrorChecker() override {}
 
   // Checks if an actionable error has been hit. Called repeatedly each time PSS
   // notifies observers of a state change.
-  virtual bool IsExitConditionSatisfied() override {
+  bool IsExitConditionSatisfied() override {
     ProfileSyncService::Status status;
     service()->QueryDetailedSyncStatus(&status);
     return (status.sync_protocol_error.action != syncer::UNKNOWN_ACTION &&
             service()->HasUnrecoverableError());
   }
 
-  virtual std::string GetDebugMessage() const override {
+  std::string GetDebugMessage() const override {
     return "ActionableErrorChecker";
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ActionableErrorChecker);
 };
 
 IN_PROC_BROWSER_TEST_F(SyncErrorTest, BirthdayErrorTest) {
   ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
 
@@ skipping 118 matching lines @@
   ASSERT_TRUE(AwaitTypeDisabled(GetSyncService(0), syncer::TYPED_URLS));
   ASSERT_TRUE(AwaitTypeDisabled(GetSyncService(0), syncer::SESSIONS));
 
   const BookmarkNode* node1 = AddFolder(0, 0, "title1");
   SetTitle(0, node1, "new_title1");
   ASSERT_TRUE(AwaitCommitActivityCompletion(GetSyncService((0))));
   // TODO(lipalani)" Verify initial sync ended for typed url is false.
 }
 
 }  // namespace