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

chrome/browser/extensions/api/push_messaging/push_messaging_canary_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 <string>
 
 #include "base/strings/string16.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/api/push_messaging/sync_setup_helper.h"
@@ skipping 56 matching lines @@
     ExtensionApiTest::SetUp();
   }
 
   void InitializeSync() {
     ASSERT_TRUE(sync_setup_helper_->InitializeSync(profile()));
   }
 
   // InProcessBrowserTest override. Destroys the sync client and sync
   // profile created by the test.  We must clean up ProfileSyncServiceHarness
   // now before the profile is cleaned up.
-  virtual void TearDownOnMainThread() override {
-    sync_setup_helper_.reset();
-  }
+  void TearDownOnMainThread() override { sync_setup_helper_.reset(); }
 
   const SyncSetupHelper* sync_setup_helper() const {
     return sync_setup_helper_.get();
   }
 
  protected:
   // Override InProcessBrowserTest. Change behavior of the default host
   // resolver to avoid DNS lookup errors, so we can make network calls.
-  virtual void SetUpInProcessBrowserTestFixture() override {
+  void SetUpInProcessBrowserTestFixture() override {
     // The resolver object lifetime is managed by sync_test_setup, not here.
     EnableDNSLookupForThisTest(
         new net::RuleBasedHostResolverProc(host_resolver()));
   }
 
-  virtual void TearDownInProcessBrowserTestFixture() override {
+  void TearDownInProcessBrowserTestFixture() override {
     DisableDNSLookupForThisTest();
   }
 
 
   // Change behavior of the default host resolver to allow DNS lookup
   // to proceed instead of being blocked by the test infrastructure.
   void EnableDNSLookupForThisTest(
       net::RuleBasedHostResolverProc* host_resolver) {
     // mock_host_resolver_override_ takes ownership of the resolver.
     scoped_refptr<net::RuleBasedHostResolverProc> resolver =
@@ skipping 56 matching lines @@
       "startTestWithCredentials('%s', '%s', '%s');",
       client_id.c_str(), client_secret.c_str(), refresh_token.c_str()));
 
   browser()->tab_strip_model()->GetActiveWebContents()->GetMainFrame()->
       ExecuteJavaScript(script_string);
 
   EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
 }
 
 }  // namespace extensions