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

chrome/browser/extensions/activity_log/ad_injection_browsertest.cc

 // Copyright 2014 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/files/file_path.h"
 #include "base/scoped_observer.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/activity_log/activity_actions.h"
@@ skipping 75 matching lines @@
     injection_type_ = Action::NO_AD_INJECTION;
     found_multiple_injections_ = false;
     enabled_ = true;
   }
 
   Action::InjectionType injection_type() const { return injection_type_; }
 
   bool found_multiple_injections() const { return found_multiple_injections_; }
 
  private:
-  virtual void OnExtensionActivity(scoped_refptr<Action> action) override;
+  void OnExtensionActivity(scoped_refptr<Action> action) override;
 
   ScopedObserver<ActivityLog, ActivityLog::Observer> scoped_observer_;
 
   // The associated BrowserContext.
   content::BrowserContext* context_;
 
   // The type of the last injection.
   Action::InjectionType injection_type_;
 
   // Whether or not we found multiple injection types (which shouldn't happen).
@@ skipping 25 matching lines @@
       found_multiple_injections_ = true;
     injection_type_ = type;
   }
 }
 
 // A mock for the AdNetworkDatabase. This simply says that the URL
 // http://www.known-ads.adnetwork is an ad network, and nothing else is.
 class TestAdNetworkDatabase : public AdNetworkDatabase {
  public:
   TestAdNetworkDatabase();
-  virtual ~TestAdNetworkDatabase();
+  ~TestAdNetworkDatabase() override;
 
  private:
-  virtual bool IsAdNetwork(const GURL& url) const override;
+  bool IsAdNetwork(const GURL& url) const override;
 
   GURL ad_network_url1_;
   GURL ad_network_url2_;
 };
 
 TestAdNetworkDatabase::TestAdNetworkDatabase() : ad_network_url1_(kAdNetwork1),
                                                  ad_network_url2_(kAdNetwork2) {
 }
 
 TestAdNetworkDatabase::~TestAdNetworkDatabase() {}
@@ skipping 10 matching lines @@
   return response.Pass();
 }
 
 }  // namespace
 
 class AdInjectionBrowserTest : public ExtensionBrowserTest {
  protected:
   AdInjectionBrowserTest();
   virtual ~AdInjectionBrowserTest();
 
-  virtual void SetUpOnMainThread() override;
-  virtual void TearDownOnMainThread() override;
+  void SetUpOnMainThread() override;
+  void TearDownOnMainThread() override;
 
   // Handle the "Reset Begin" stage of the test.
   testing::AssertionResult HandleResetBeginStage();
 
   // Handle the "Reset End" stage of the test.
   testing::AssertionResult HandleResetEndStage();
 
   // Handle the "Testing" stage of the test.
   testing::AssertionResult HandleTestingStage(const std::string& message);
 
@@ skipping 190 matching lines @@
     // In all cases (except for "Test Complete", in which case we already
     // break'ed), we reply with a continue message.
     listener()->Reply("Continue");
     listener()->Reset();
   }
 }
 
 // If this test grows, we should consolidate it and AdInjectionBrowserTest.
 class ExecuteScriptAdInjectionBrowserTest : public ExtensionBrowserTest {
  protected:
-  virtual void SetUpOnMainThread() override;
-  virtual void TearDownOnMainThread() override;
+  void SetUpOnMainThread() override;
+  void TearDownOnMainThread() override;
 };
 
 void ExecuteScriptAdInjectionBrowserTest::SetUpOnMainThread() {
   ExtensionBrowserTest::SetUpOnMainThread();
 
   ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
   embedded_test_server()->RegisterRequestHandler(base::Bind(&HandleRequest));
 
   // Enable the activity log for this test.
   ActivityLog::GetInstance(profile())->SetWatchdogAppActiveForTesting(true);
@@ skipping 32 matching lines @@
   // We should have injected an ad.
   EXPECT_EQ(Action::INJECTION_NEW_AD, observer.injection_type());
   EXPECT_FALSE(observer.found_multiple_injections());
 }
 
 // TODO(rdevlin.cronin): We test a good amount of ways of injecting ads with
 // the above test, but more is better in testing.
 // See crbug.com/357204.
 
 }  // namespace extensions