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

chrome/browser/external_protocol/external_protocol_handler_unittest.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 "chrome/browser/external_protocol/external_protocol_handler.h"
 
 #include "base/message_loop/message_loop.h"
 #include "content/public/test/test_browser_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::BrowserThread;
 
 class FakeExternalProtocolHandlerWorker
     : public ShellIntegration::DefaultProtocolClientWorker {
  public:
   FakeExternalProtocolHandlerWorker(
       ShellIntegration::DefaultWebClientObserver* observer,
       const std::string& protocol,
       ShellIntegration::DefaultWebClientState os_state)
       : ShellIntegration::DefaultProtocolClientWorker(observer, protocol),
         os_state_(os_state) {}
 
  private:
-  virtual ~FakeExternalProtocolHandlerWorker() {}
+  ~FakeExternalProtocolHandlerWorker() override {}
 
-  virtual ShellIntegration::DefaultWebClientState CheckIsDefault() override {
+  ShellIntegration::DefaultWebClientState CheckIsDefault() override {
     return os_state_;
   }
 
-  virtual bool SetAsDefault(bool interactive_permitted) override {
-    return true;
-  }
+  bool SetAsDefault(bool interactive_permitted) override { return true; }
 
   ShellIntegration::DefaultWebClientState os_state_;
 };
 
 class FakeExternalProtocolHandlerDelegate
     : public ExternalProtocolHandler::Delegate {
  public:
   FakeExternalProtocolHandlerDelegate()
       : block_state_(ExternalProtocolHandler::BLOCK),
         os_state_(ShellIntegration::UNKNOWN_DEFAULT),
         has_launched_(false),
         has_prompted_(false),
         has_blocked_ (false) {}
 
-  virtual ShellIntegration::DefaultProtocolClientWorker* CreateShellWorker(
+  ShellIntegration::DefaultProtocolClientWorker* CreateShellWorker(
       ShellIntegration::DefaultWebClientObserver* observer,
       const std::string& protocol) override {
     return new FakeExternalProtocolHandlerWorker(observer, protocol, os_state_);
   }
 
-  virtual ExternalProtocolHandler::BlockState GetBlockState(
+  ExternalProtocolHandler::BlockState GetBlockState(
       const std::string& scheme) override {
     return block_state_;
   }
 
-  virtual void BlockRequest() override {
+  void BlockRequest() override {
     ASSERT_TRUE(block_state_ == ExternalProtocolHandler::BLOCK ||
                 os_state_ == ShellIntegration::IS_DEFAULT);
     has_blocked_ = true;
   }
 
-  virtual void RunExternalProtocolDialog(const GURL& url,
-                                         int render_process_host_id,
-                                         int routing_id) override {
+  void RunExternalProtocolDialog(const GURL& url,
+                                 int render_process_host_id,
+                                 int routing_id) override {
     ASSERT_EQ(block_state_, ExternalProtocolHandler::UNKNOWN);
     ASSERT_NE(os_state_, ShellIntegration::IS_DEFAULT);
     has_prompted_ = true;
   }
 
-  virtual void LaunchUrlWithoutSecurityCheck(const GURL& url) override {
+  void LaunchUrlWithoutSecurityCheck(const GURL& url) override {
     ASSERT_EQ(block_state_, ExternalProtocolHandler::DONT_BLOCK);
     ASSERT_NE(os_state_, ShellIntegration::IS_DEFAULT);
     has_launched_ = true;
   }
 
-  virtual void FinishedProcessingCheck() override {
+  void FinishedProcessingCheck() override {
     base::MessageLoop::current()->Quit();
   }
 
   void set_os_state(ShellIntegration::DefaultWebClientState value) {
     os_state_ = value;
   }
 
   void set_block_state(ExternalProtocolHandler::BlockState value) {
     block_state_ = value;
   }
@@ skipping 97 matching lines @@
   DoTest(ExternalProtocolHandler::UNKNOWN,
          ShellIntegration::NOT_DEFAULT,
          true, false, false);
 }
 
 TEST_F(ExternalProtocolHandlerTest, TestLaunchSchemeUnknownChromeUnknown) {
   DoTest(ExternalProtocolHandler::UNKNOWN,
          ShellIntegration::UNKNOWN_DEFAULT,
          true, false, false);
 }