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

The Google C++ style guide states:

  Explicitly annotate overrides of virtual functions or virtual
  destructors with an override or (less frequently) final specifier.
  Older (pre-C++11) code will use the virtual keyword as an inferior
  alternative annotation. For clarity, use exactly one of override,
  final, or virtual when declaring an override.

To better conform to these guidelines, the following constructs have
been rewritten:

- if a base class has a virtual destructor, then:
    virtual ~Foo();                   ->  ~Foo() override;
- virtual void Foo() override;        ->  void Foo() override;
- virtual void Foo() override final;  ->  void Foo() final;

This patch was automatically generated. The clang plugin can generate
fixit hints, which are suggested edits when it is 100% sure it knows how
to fix a problem. The hints from the clang plugin were applied to the
source tree using the tool in https://codereview.chromium.org/598073004.

BUG=417463
R=mkwst@chromium.org

Review URL: https://codereview.chromium.org/666123002

Cr-Commit-Position: refs/heads/master@{#300575}

chrome/browser/browsing_data/browsing_data_appcache_helper.h

@@ -76,11 +76,11 @@ class CannedBrowsingDataAppCacheHelper : public BrowsingDataAppCacheHelper {
   const OriginAppCacheInfoMap& GetOriginAppCacheInfoMap() const;
 
   // BrowsingDataAppCacheHelper methods.
-  virtual void StartFetching(const base::Closure& completion_callback) override;
-  virtual void DeleteAppCacheGroup(const GURL& manifest_url) override;
+  void StartFetching(const base::Closure& completion_callback) override;
+  void DeleteAppCacheGroup(const GURL& manifest_url) override;
 
  private:
-  virtual ~CannedBrowsingDataAppCacheHelper();
+  ~CannedBrowsingDataAppCacheHelper() override;
 
   DISALLOW_COPY_AND_ASSIGN(CannedBrowsingDataAppCacheHelper);
 };

chrome/browser/browsing_data/browsing_data_channel_id_helper.cc

@@ -24,11 +24,11 @@ class BrowsingDataChannelIDHelperImpl
       net::URLRequestContextGetter* request_context);
 
   // BrowsingDataChannelIDHelper methods.
-  virtual void StartFetching(const FetchResultCallback& callback) override;
-  virtual void DeleteChannelID(const std::string& server_id) override;
+  void StartFetching(const FetchResultCallback& callback) override;
+  void DeleteChannelID(const std::string& server_id) override;
 
  private:
-  virtual ~BrowsingDataChannelIDHelperImpl();
+  ~BrowsingDataChannelIDHelperImpl() override;
 
   // Fetch the certs. This must be called in the IO thread.
   void FetchOnIOThread();

chrome/browser/browsing_data/browsing_data_channel_id_helper.h

@@ -68,11 +68,11 @@ class CannedBrowsingDataChannelIDHelper
   size_t GetChannelIDCount() const;
 
   // BrowsingDataChannelIDHelper methods.
-  virtual void StartFetching(const FetchResultCallback& callback) override;
-  virtual void DeleteChannelID(const std::string& server_id) override;
+  void StartFetching(const FetchResultCallback& callback) override;
+  void DeleteChannelID(const std::string& server_id) override;
 
  private:
-  virtual ~CannedBrowsingDataChannelIDHelper();
+  ~CannedBrowsingDataChannelIDHelper() override;
 
   void FinishFetching();
 

chrome/browser/browsing_data/browsing_data_channel_id_helper_unittest.cc

@@ -53,9 +53,7 @@ class BrowsingDataChannelIDHelperTest
   }
 
   // net::SSLConfigService::Observer implementation:
-  virtual void OnSSLConfigChanged() override {
-    ssl_config_changed_count_++;
-  }
+  void OnSSLConfigChanged() override { ssl_config_changed_count_++; }
 
  protected:
   content::TestBrowserThreadBundle thread_bundle_;

chrome/browser/browsing_data/browsing_data_cookie_helper.h

@@ -126,9 +126,9 @@ class CannedBrowsingDataCookieHelper : public BrowsingDataCookieHelper {
   bool empty() const;
 
   // BrowsingDataCookieHelper methods.
-  virtual void StartFetching(
+  void StartFetching(
       const net::CookieMonster::GetCookieListCallback& callback) override;
-  virtual void DeleteCookie(const net::CanonicalCookie& cookie) override;
+  void DeleteCookie(const net::CanonicalCookie& cookie) override;
 
   // Returns the number of stored cookies.
   size_t GetCookieCount() const;
@@ -145,7 +145,7 @@ class CannedBrowsingDataCookieHelper : public BrowsingDataCookieHelper {
   bool DeleteMatchingCookie(const net::CanonicalCookie& add_cookie,
                             canonical_cookie::CookieHashSet* cookie_set);
 
-  virtual ~CannedBrowsingDataCookieHelper();
+  ~CannedBrowsingDataCookieHelper() override;
 
   // Returns the |CookieSet| for the given |origin|.
   canonical_cookie::CookieHashSet* GetCookiesFor(const GURL& origin);

chrome/browser/browsing_data/browsing_data_database_helper.h

@@ -134,14 +134,13 @@ class CannedBrowsingDataDatabaseHelper : public BrowsingDataDatabaseHelper {
   const std::set<PendingDatabaseInfo>& GetPendingDatabaseInfo();
 
   // BrowsingDataDatabaseHelper implementation.
-  virtual void StartFetching(
-      const base::Callback<void(const std::list<DatabaseInfo>&)>& callback)
-          override;
-  virtual void DeleteDatabase(const std::string& origin_identifier,
-                              const std::string& name) override;
+  void StartFetching(const base::Callback<void(const std::list<DatabaseInfo>&)>&
+                         callback) override;
+  void DeleteDatabase(const std::string& origin_identifier,
+                      const std::string& name) override;
 
  private:
-  virtual ~CannedBrowsingDataDatabaseHelper();
+  ~CannedBrowsingDataDatabaseHelper() override;
 
   std::set<PendingDatabaseInfo> pending_database_info_;
 

chrome/browser/browsing_data/browsing_data_file_system_helper.cc

@@ -34,12 +34,13 @@ class BrowsingDataFileSystemHelperImpl : public BrowsingDataFileSystemHelper {
   // BrowsingDataFileSystemHelper implementation
   explicit BrowsingDataFileSystemHelperImpl(
       storage::FileSystemContext* filesystem_context);
-  virtual void StartFetching(const base::Callback<
-      void(const std::list<FileSystemInfo>&)>& callback) override;
-  virtual void DeleteFileSystemOrigin(const GURL& origin) override;
+  void StartFetching(
+      const base::Callback<void(const std::list<FileSystemInfo>&)>& callback)
+      override;
+  void DeleteFileSystemOrigin(const GURL& origin) override;
 
  private:
-  virtual ~BrowsingDataFileSystemHelperImpl();
+  ~BrowsingDataFileSystemHelperImpl() override;
 
   // Enumerates all filesystem files, storing the resulting list into
   // file_system_file_ for later use. This must be called on the file

chrome/browser/browsing_data/browsing_data_file_system_helper.h

@@ -120,17 +120,18 @@ class CannedBrowsingDataFileSystemHelper
   }
 
   // BrowsingDataFileSystemHelper implementation.
-  virtual void StartFetching(const base::Callback<
-      void(const std::list<FileSystemInfo>&)>& callback) override;
+  void StartFetching(
+      const base::Callback<void(const std::list<FileSystemInfo>&)>& callback)
+      override;
 
   // Note that this doesn't actually have an implementation for this canned
   // class. It hasn't been necessary for anything that uses the canned
   // implementation, as the canned class is only used in tests, or in read-only
   // contexts (like the non-modal cookie dialog).
-  virtual void DeleteFileSystemOrigin(const GURL& origin) override {}
+  void DeleteFileSystemOrigin(const GURL& origin) override {}
 
  private:
-  virtual ~CannedBrowsingDataFileSystemHelper();
+  ~CannedBrowsingDataFileSystemHelper() override;
 
   // Holds the current list of filesystems returned to the client.
   std::list<FileSystemInfo> file_system_info_;

chrome/browser/browsing_data/browsing_data_flash_lso_helper.cc

@@ -21,20 +21,17 @@ class BrowsingDataFlashLSOHelperImpl
       content::BrowserContext* browser_context);
 
   // BrowsingDataFlashLSOHelper implementation:
-  virtual void StartFetching(
-      const GetSitesWithFlashDataCallback& callback) override;
-  virtual void DeleteFlashLSOsForSite(const std::string& site) override;
+  void StartFetching(const GetSitesWithFlashDataCallback& callback) override;
+  void DeleteFlashLSOsForSite(const std::string& site) override;
 
   // PepperFlashSettingsManager::Client overrides:
-  virtual void OnGetSitesWithDataCompleted(
+  void OnGetSitesWithDataCompleted(
       uint32 request_id,
       const std::vector<std::string>& sites) override;
-  virtual void OnClearSiteDataCompleted(
-      uint32 request_id,
-      bool success) override;
+  void OnClearSiteDataCompleted(uint32 request_id, bool success) override;
 
  private:
-  virtual ~BrowsingDataFlashLSOHelperImpl();
+  ~BrowsingDataFlashLSOHelperImpl() override;
 
   // Asynchronously fetches and deletes data and calls us back.
   PepperFlashSettingsManager settings_manager_;

chrome/browser/browsing_data/browsing_data_indexed_db_helper.h

@@ -115,13 +115,12 @@ class CannedBrowsingDataIndexedDBHelper
       GetIndexedDBInfo() const;
 
   // BrowsingDataIndexedDBHelper methods.
-  virtual void StartFetching(
-      const base::Callback<void(const std::list<content::IndexedDBInfo>&)>&
-          callback) override;
-  virtual void DeleteIndexedDB(const GURL& origin) override;
+  void StartFetching(const base::Callback<
+      void(const std::list<content::IndexedDBInfo>&)>& callback) override;
+  void DeleteIndexedDB(const GURL& origin) override;
 
  private:
-  virtual ~CannedBrowsingDataIndexedDBHelper();
+  ~CannedBrowsingDataIndexedDBHelper() override;
 
   std::set<PendingIndexedDBInfo> pending_indexed_db_info_;
 

chrome/browser/browsing_data/browsing_data_local_storage_helper.h

@@ -92,13 +92,13 @@ class CannedBrowsingDataLocalStorageHelper
   const std::set<GURL>& GetLocalStorageInfo() const;
 
   // BrowsingDataLocalStorageHelper implementation.
-  virtual void StartFetching(
+  void StartFetching(
       const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback)
-          override;
-  virtual void DeleteOrigin(const GURL& origin) override;
+      override;
+  void DeleteOrigin(const GURL& origin) override;
 
  private:
-  virtual ~CannedBrowsingDataLocalStorageHelper();
+  ~CannedBrowsingDataLocalStorageHelper() override;
 
   std::set<GURL> pending_local_storage_info_;
 

chrome/browser/browsing_data/browsing_data_quota_helper_impl.h

@@ -27,14 +27,14 @@ class QuotaManager;
 // IO thread, we have to communicate over thread using PostTask.
 class BrowsingDataQuotaHelperImpl : public BrowsingDataQuotaHelper {
  public:
-  virtual void StartFetching(const FetchResultCallback& callback) override;
-  virtual void RevokeHostQuota(const std::string& host) override;
+  void StartFetching(const FetchResultCallback& callback) override;
+  void RevokeHostQuota(const std::string& host) override;
 
  private:
   BrowsingDataQuotaHelperImpl(base::MessageLoopProxy* ui_thread,
                               base::MessageLoopProxy* io_thread,
                               storage::QuotaManager* quota_manager);
-  virtual ~BrowsingDataQuotaHelperImpl();
+  ~BrowsingDataQuotaHelperImpl() override;
 
   void FetchQuotaInfo();
 

chrome/browser/browsing_data/browsing_data_remover.h

@@ -250,7 +250,14 @@ class BrowsingDataRemover
   // to be deleted by other objects so make destructor private and DeleteHelper
   // a friend.
   friend class base::DeleteHelper<BrowsingDataRemover>;
-  virtual ~BrowsingDataRemover();
+
+  // When plugins aren't enabled, there is no base class, so adding an override
+  // specifier would result in a compile error.
+#if defined(ENABLE_PLUGINS)
+  ~BrowsingDataRemover() override;
+#else
+  ~BrowsingDataRemover();
+#endif
 
   // Callback for when TemplateURLService has finished loading. Clears the data,
   // clears the respective waiting flag, and invokes NotifyAndDeleteIfDone.
@@ -261,8 +268,8 @@ class BrowsingDataRemover
 
 #if defined(ENABLE_PLUGINS)
   // PepperFlashSettingsManager::Client implementation.
-  virtual void OnDeauthorizeContentLicensesCompleted(uint32 request_id,
-                                                     bool success) override;
+  void OnDeauthorizeContentLicensesCompleted(uint32 request_id,
+                                             bool success) override;
 #endif
 
 #if defined (OS_CHROMEOS)

chrome/browser/browsing_data/browsing_data_remover_browsertest.cc

@@ -38,7 +38,7 @@ class BrowsingDataRemoverBrowserTest : public InProcessBrowserTest {
  public:
   BrowsingDataRemoverBrowserTest() {}
 
-  virtual void SetUpOnMainThread() override {
+  void SetUpOnMainThread() override {
     base::FilePath path;
     PathService::Get(content::DIR_TEST_DATA, &path);
     BrowserThread::PostTask(

chrome/browser/browsing_data/browsing_data_remover_test_util.h

@@ -14,13 +14,13 @@ class BrowsingDataRemoverCompletionObserver
     : public BrowsingDataRemover::Observer {
  public:
   explicit BrowsingDataRemoverCompletionObserver(BrowsingDataRemover* remover);
-  virtual ~BrowsingDataRemoverCompletionObserver();
+  ~BrowsingDataRemoverCompletionObserver() override;
 
   void BlockUntilCompletion();
 
  protected:
   // BrowsingDataRemover::Observer:
-  virtual void OnBrowsingDataRemoverDone() override;
+  void OnBrowsingDataRemoverDone() override;
 
  private:
   scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
@@ -32,14 +32,14 @@ class BrowsingDataRemoverCompletionInhibitor
     : public BrowsingDataRemover::CompletionInhibitor {
  public:
   BrowsingDataRemoverCompletionInhibitor();
-  virtual ~BrowsingDataRemoverCompletionInhibitor();
+  ~BrowsingDataRemoverCompletionInhibitor() override;
 
   void BlockUntilNearCompletion();
   void ContinueToCompletion();
 
  protected:
   // BrowsingDataRemover::CompletionInhibitor:
-  virtual void OnBrowsingDataRemoverWouldComplete(
+  void OnBrowsingDataRemoverWouldComplete(
       BrowsingDataRemover* remover,
       const base::Closure& continue_to_completion) override;
 

chrome/browser/browsing_data/browsing_data_remover_unittest.cc

@@ -133,41 +133,29 @@ struct StoragePartitionRemovalData {
 class TestStoragePartition : public StoragePartition {
  public:
   TestStoragePartition() {}
-  virtual ~TestStoragePartition() {}
+  ~TestStoragePartition() override {}
 
   // content::StoragePartition implementation.
-  virtual base::FilePath GetPath() override { return base::FilePath(); }
-  virtual net::URLRequestContextGetter* GetURLRequestContext() override {
+  base::FilePath GetPath() override { return base::FilePath(); }
+  net::URLRequestContextGetter* GetURLRequestContext() override { return NULL; }
+  net::URLRequestContextGetter* GetMediaURLRequestContext() override {
     return NULL;
   }
-  virtual net::URLRequestContextGetter* GetMediaURLRequestContext() override {
-    return NULL;
-  }
-  virtual storage::QuotaManager* GetQuotaManager() override { return NULL; }
-  virtual content::AppCacheService* GetAppCacheService() override {
-    return NULL;
-  }
-  virtual storage::FileSystemContext* GetFileSystemContext() override {
-    return NULL;
-  }
-  virtual storage::DatabaseTracker* GetDatabaseTracker() override {
-    return NULL;
-  }
-  virtual content::DOMStorageContext* GetDOMStorageContext() override {
-    return NULL;
-  }
-  virtual content::IndexedDBContext* GetIndexedDBContext() override {
-    return NULL;
-  }
-  virtual content::ServiceWorkerContext* GetServiceWorkerContext() override {
+  storage::QuotaManager* GetQuotaManager() override { return NULL; }
+  content::AppCacheService* GetAppCacheService() override { return NULL; }
+  storage::FileSystemContext* GetFileSystemContext() override { return NULL; }
+  storage::DatabaseTracker* GetDatabaseTracker() override { return NULL; }
+  content::DOMStorageContext* GetDOMStorageContext() override { return NULL; }
+  content::IndexedDBContext* GetIndexedDBContext() override { return NULL; }
+  content::ServiceWorkerContext* GetServiceWorkerContext() override {
     return NULL;
   }
 
-  virtual void ClearDataForOrigin(uint32 remove_mask,
-                                  uint32 quota_storage_remove_mask,
-                                  const GURL& storage_origin,
-                                  net::URLRequestContextGetter* rq_context,
-                                  const base::Closure& callback) override {
+  void ClearDataForOrigin(uint32 remove_mask,
+                          uint32 quota_storage_remove_mask,
+                          const GURL& storage_origin,
+                          net::URLRequestContextGetter* rq_context,
+                          const base::Closure& callback) override {
     BrowserThread::PostTask(BrowserThread::UI,
                             FROM_HERE,
                             base::Bind(&TestStoragePartition::AsyncRunCallback,
@@ -175,13 +163,13 @@ class TestStoragePartition : public StoragePartition {
                                        callback));
   }
 
-  virtual void ClearData(uint32 remove_mask,
-                         uint32 quota_storage_remove_mask,
-                         const GURL& storage_origin,
-                         const OriginMatcherFunction& origin_matcher,
-                         const base::Time begin,
-                         const base::Time end,
-                         const base::Closure& callback) override {
+  void ClearData(uint32 remove_mask,
+                 uint32 quota_storage_remove_mask,
+                 const GURL& storage_origin,
+                 const OriginMatcherFunction& origin_matcher,
+                 const base::Time begin,
+                 const base::Time end,
+                 const base::Closure& callback) override {
     // Store stuff to verify parameters' correctness later.
     storage_partition_removal_data_.remove_mask = remove_mask;
     storage_partition_removal_data_.quota_storage_remove_mask =
@@ -317,7 +305,7 @@ class RemoveChannelIDTester : public net::SSLConfigService::Observer {
     ssl_config_service_->AddObserver(this);
   }
 
-  virtual ~RemoveChannelIDTester() {
+  ~RemoveChannelIDTester() override {
     ssl_config_service_->RemoveObserver(this);
   }
 
@@ -361,9 +349,7 @@ class RemoveChannelIDTester : public net::SSLConfigService::Observer {
   }
 
   // net::SSLConfigService::Observer implementation:
-  virtual void OnSSLConfigChanged() override {
-    ssl_config_changed_count_++;
-  }
+  void OnSSLConfigChanged() override { ssl_config_changed_count_++; }
 
  private:
   static void GetAllChannelIDsCallback(
@@ -441,7 +427,7 @@ class RemoveAutofillTester : public autofill::PersonalDataManagerObserver {
     personal_data_manager_->AddObserver(this);
   }
 
-  virtual ~RemoveAutofillTester() {
+  ~RemoveAutofillTester() override {
     personal_data_manager_->RemoveObserver(this);
   }
 
@@ -512,7 +498,7 @@ class RemoveAutofillTester : public autofill::PersonalDataManagerObserver {
   }
 
  private:
-  virtual void OnPersonalDataChanged() override {
+  void OnPersonalDataChanged() override {
     base::MessageLoop::current()->Quit();
   }
 
@@ -597,25 +583,24 @@ class MockDomainReliabilityService : public DomainReliabilityService {
  public:
   MockDomainReliabilityService() : clear_count_(0) {}
 
-  virtual ~MockDomainReliabilityService() {}
+  ~MockDomainReliabilityService() override {}
 
-  virtual scoped_ptr<DomainReliabilityMonitor> CreateMonitor(
+  scoped_ptr<DomainReliabilityMonitor> CreateMonitor(
       scoped_refptr<base::SingleThreadTaskRunner> network_task_runner)
       override {
     NOTREACHED();
     return scoped_ptr<DomainReliabilityMonitor>();
   }
 
-  virtual void ClearBrowsingData(DomainReliabilityClearMode clear_mode,
-                                 const base::Closure& callback) override {
+  void ClearBrowsingData(DomainReliabilityClearMode clear_mode,
+                         const base::Closure& callback) override {
     clear_count_++;
     last_clear_mode_ = clear_mode;
     callback.Run();
   }
 
-  virtual void GetWebUIData(
-      const base::Callback<void(scoped_ptr<base::Value>)>& callback)
-      const override {
+  void GetWebUIData(const base::Callback<void(scoped_ptr<base::Value>)>&
+                        callback) const override {
     NOTREACHED();
   }
 
@@ -638,7 +623,7 @@ struct TestingDomainReliabilityServiceFactoryUserData
       : context(context),
         service(service),
         attached(false) {}
-  virtual ~TestingDomainReliabilityServiceFactoryUserData() {}
+  ~TestingDomainReliabilityServiceFactoryUserData() override {}
 
   content::BrowserContext* const context;
   MockDomainReliabilityService* const service;
@@ -801,9 +786,9 @@ class BrowsingDataRemoverTest : public testing::Test,
   }
 
   // content::NotificationObserver implementation.
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) override {
+  void Observe(int type,
+               const content::NotificationSource& source,
+               const content::NotificationDetails& details) override {
     DCHECK_EQ(type, chrome::NOTIFICATION_BROWSING_DATA_REMOVED);
 
     // We're not taking ownership of the details object, but storing a copy of

chrome/browser/browsing_data/browsing_data_service_worker_helper.h

@@ -126,12 +126,12 @@ class CannedBrowsingDataServiceWorkerHelper
       GetServiceWorkerUsageInfo() const;
 
   // BrowsingDataServiceWorkerHelper methods.
-  virtual void StartFetching(const base::Callback<void(
+  void StartFetching(const base::Callback<void(
       const std::list<content::ServiceWorkerUsageInfo>&)>& callback) override;
-  virtual void DeleteServiceWorkers(const GURL& origin) override;
+  void DeleteServiceWorkers(const GURL& origin) override;
 
  private:
-  virtual ~CannedBrowsingDataServiceWorkerHelper();
+  ~CannedBrowsingDataServiceWorkerHelper() override;
 
   std::set<PendingServiceWorkerUsageInfo> pending_service_worker_info_;
 

chrome/browser/browsing_data/mock_browsing_data_appcache_helper.h

@@ -14,11 +14,11 @@ class MockBrowsingDataAppCacheHelper
   explicit MockBrowsingDataAppCacheHelper(
       content::BrowserContext* browser_context);
 
-  virtual void StartFetching(const base::Closure& completion_callback) override;
-  virtual void DeleteAppCacheGroup(const GURL& manifest_url) override;
+  void StartFetching(const base::Closure& completion_callback) override;
+  void DeleteAppCacheGroup(const GURL& manifest_url) override;
 
  private:
-  virtual ~MockBrowsingDataAppCacheHelper();
+  ~MockBrowsingDataAppCacheHelper() override;
 
   DISALLOW_COPY_AND_ASSIGN(MockBrowsingDataAppCacheHelper);
 };

chrome/browser/browsing_data/mock_browsing_data_channel_id_helper.h

@@ -17,8 +17,8 @@ class MockBrowsingDataChannelIDHelper
   MockBrowsingDataChannelIDHelper();
 
   // BrowsingDataChannelIDHelper methods.
-  virtual void StartFetching(const FetchResultCallback& callback) override;
-  virtual void DeleteChannelID(const std::string& server_id) override;
+  void StartFetching(const FetchResultCallback& callback) override;
+  void DeleteChannelID(const std::string& server_id) override;
 
   // Adds a channel_id sample.
   void AddChannelIDSample(const std::string& server_id);
@@ -34,7 +34,7 @@ class MockBrowsingDataChannelIDHelper
   bool AllDeleted();
 
  private:
-  virtual ~MockBrowsingDataChannelIDHelper();
+  ~MockBrowsingDataChannelIDHelper() override;
 
   FetchResultCallback callback_;
 

chrome/browser/browsing_data/mock_browsing_data_cookie_helper.h

@@ -18,9 +18,9 @@ class MockBrowsingDataCookieHelper : public BrowsingDataCookieHelper {
       net::URLRequestContextGetter* request_context_getter);
 
   // BrowsingDataCookieHelper methods.
-  virtual void StartFetching(
-      const net::CookieMonster::GetCookieListCallback &callback) override;
-  virtual void DeleteCookie(const net::CanonicalCookie& cookie) override;
+  void StartFetching(
+      const net::CookieMonster::GetCookieListCallback& callback) override;
+  void DeleteCookie(const net::CanonicalCookie& cookie) override;
 
   // Adds some cookie samples.
   void AddCookieSamples(const GURL& url, const std::string& cookie_line);
@@ -36,7 +36,7 @@ class MockBrowsingDataCookieHelper : public BrowsingDataCookieHelper {
   bool AllDeleted();
 
  private:
-  virtual ~MockBrowsingDataCookieHelper();
+  ~MockBrowsingDataCookieHelper() override;
 
   net::CookieMonster::GetCookieListCallback callback_;
 

chrome/browser/browsing_data/mock_browsing_data_database_helper.h

@@ -19,12 +19,11 @@ class MockBrowsingDataDatabaseHelper : public BrowsingDataDatabaseHelper {
  public:
   explicit MockBrowsingDataDatabaseHelper(Profile* profile);
 
-  virtual void StartFetching(
-      const base::Callback<void(const std::list<DatabaseInfo>&)>& callback)
-          override;
+  void StartFetching(const base::Callback<void(const std::list<DatabaseInfo>&)>&
+                         callback) override;
 
-  virtual void DeleteDatabase(const std::string& origin,
-      const std::string& name) override;
+  void DeleteDatabase(const std::string& origin,
+                      const std::string& name) override;
 
   // Adds some DatabaseInfo samples.
   void AddDatabaseSamples();
@@ -44,7 +43,7 @@ class MockBrowsingDataDatabaseHelper : public BrowsingDataDatabaseHelper {
   std::string last_deleted_db_;
 
  private:
-  virtual ~MockBrowsingDataDatabaseHelper();
+  ~MockBrowsingDataDatabaseHelper() override;
 
   base::Callback<void(const std::list<DatabaseInfo>&)> callback_;
 

chrome/browser/browsing_data/mock_browsing_data_file_system_helper.h

@@ -21,9 +21,10 @@ class MockBrowsingDataFileSystemHelper : public BrowsingDataFileSystemHelper {
   explicit MockBrowsingDataFileSystemHelper(Profile* profile);
 
   // BrowsingDataFileSystemHelper implementation.
-  virtual void StartFetching(const base::Callback<
-      void(const std::list<FileSystemInfo>&)>& callback) override;
-  virtual void DeleteFileSystemOrigin(const GURL& origin) override;
+  void StartFetching(
+      const base::Callback<void(const std::list<FileSystemInfo>&)>& callback)
+      override;
+  void DeleteFileSystemOrigin(const GURL& origin) override;
 
   // Adds a specific filesystem.
   void AddFileSystem(const GURL& origin,
@@ -47,7 +48,7 @@ class MockBrowsingDataFileSystemHelper : public BrowsingDataFileSystemHelper {
   GURL last_deleted_origin_;
 
  private:
-  virtual ~MockBrowsingDataFileSystemHelper();
+  ~MockBrowsingDataFileSystemHelper() override;
 
   base::Callback<void(const std::list<FileSystemInfo>&)> callback_;
 

chrome/browser/browsing_data/mock_browsing_data_flash_lso_helper.h

@@ -17,9 +17,8 @@ class MockBrowsingDataFlashLSOHelper : public BrowsingDataFlashLSOHelper {
       content::BrowserContext* browser_context);
 
   // BrowsingDataFlashLSOHelper implementation:
-  virtual void StartFetching(
-      const GetSitesWithFlashDataCallback& callback) override;
-  virtual void DeleteFlashLSOsForSite(const std::string& site) override;
+  void StartFetching(const GetSitesWithFlashDataCallback& callback) override;
+  void DeleteFlashLSOsForSite(const std::string& site) override;
 
   // Adds a domain sample.
   void AddFlashLSODomain(const std::string& domain);
@@ -31,7 +30,7 @@ class MockBrowsingDataFlashLSOHelper : public BrowsingDataFlashLSOHelper {
   bool AllDeleted();
 
  private:
-  virtual ~MockBrowsingDataFlashLSOHelper();
+  ~MockBrowsingDataFlashLSOHelper() override;
 
   GetSitesWithFlashDataCallback callback_;
 

chrome/browser/browsing_data/mock_browsing_data_indexed_db_helper.h

@@ -36,13 +36,12 @@ class MockBrowsingDataIndexedDBHelper
   bool AllDeleted();
 
   // BrowsingDataIndexedDBHelper.
-  virtual void StartFetching(
-      const base::Callback<void(const std::list<content::IndexedDBInfo>&)>&
-          callback) override;
-  virtual void DeleteIndexedDB(const GURL& origin) override;
+  void StartFetching(const base::Callback<
+      void(const std::list<content::IndexedDBInfo>&)>& callback) override;
+  void DeleteIndexedDB(const GURL& origin) override;
 
  private:
-  virtual ~MockBrowsingDataIndexedDBHelper();
+  ~MockBrowsingDataIndexedDBHelper() override;
 
   base::Callback<void(const std::list<content::IndexedDBInfo>&)> callback_;
   std::map<GURL, bool> origins_;

chrome/browser/browsing_data/mock_browsing_data_local_storage_helper.h

@@ -21,10 +21,10 @@ class MockBrowsingDataLocalStorageHelper
   explicit MockBrowsingDataLocalStorageHelper(Profile* profile);
 
   // BrowsingDataLocalStorageHelper implementation.
-  virtual void StartFetching(
+  void StartFetching(
       const base::Callback<void(const std::list<LocalStorageInfo>&)>& callback)
-          override;
-  virtual void DeleteOrigin(const GURL& origin) override;
+      override;
+  void DeleteOrigin(const GURL& origin) override;
 
   // Adds some LocalStorageInfo samples.
   void AddLocalStorageSamples();
@@ -42,7 +42,7 @@ class MockBrowsingDataLocalStorageHelper
   GURL last_deleted_origin_;
 
  private:
-  virtual ~MockBrowsingDataLocalStorageHelper();
+  ~MockBrowsingDataLocalStorageHelper() override;
 
   base::Callback<void(const std::list<LocalStorageInfo>&)> callback_;
 

chrome/browser/browsing_data/mock_browsing_data_quota_helper.h

@@ -15,8 +15,8 @@ class MockBrowsingDataQuotaHelper : public BrowsingDataQuotaHelper {
  public:
   explicit MockBrowsingDataQuotaHelper(Profile* profile);
 
-  virtual void StartFetching(const FetchResultCallback& callback) override;
-  virtual void RevokeHostQuota(const std::string& host) override;
+  void StartFetching(const FetchResultCallback& callback) override;
+  void RevokeHostQuota(const std::string& host) override;
 
   void AddHost(const std::string& host,
                int64 temporary_usage,
@@ -26,7 +26,7 @@ class MockBrowsingDataQuotaHelper : public BrowsingDataQuotaHelper {
   void Notify();
 
  private:
-  virtual ~MockBrowsingDataQuotaHelper();
+  ~MockBrowsingDataQuotaHelper() override;
 
   FetchResultCallback callback_;
   std::list<QuotaInfo> response_;

chrome/browser/browsing_data/mock_browsing_data_service_worker_helper.h

@@ -36,12 +36,12 @@ class MockBrowsingDataServiceWorkerHelper
   bool AllDeleted();
 
   // BrowsingDataServiceWorkerHelper.
-  virtual void StartFetching(const base::Callback<void(
+  void StartFetching(const base::Callback<void(
       const std::list<content::ServiceWorkerUsageInfo>&)>& callback) override;
-  virtual void DeleteServiceWorkers(const GURL& origin) override;
+  void DeleteServiceWorkers(const GURL& origin) override;
 
  private:
-  virtual ~MockBrowsingDataServiceWorkerHelper();
+  ~MockBrowsingDataServiceWorkerHelper() override;
 
   base::Callback<void(const std::list<content::ServiceWorkerUsageInfo>&)>
       callback_;