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

chrome/browser/extensions/external_policy_loader_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 <set>
 #include <string>
 
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/values.h"
@@ skipping 52 matching lines @@
         Manifest::EXTERNAL_POLICY_DOWNLOAD,
         Extension::NO_FLAGS));
 
     // Extensions will be removed from this list as they visited,
     // so it should be emptied by the end.
     expected_extensions_ = expected_extensions;
     provider_->VisitRegisteredExtension();
     EXPECT_TRUE(expected_extensions_.empty());
   }
 
-  virtual bool OnExternalExtensionFileFound(const std::string& id,
-                                            const Version* version,
-                                            const base::FilePath& path,
-                                            Manifest::Location unused,
-                                            int unused2,
-                                            bool unused3) override {
+  bool OnExternalExtensionFileFound(const std::string& id,
+                                    const Version* version,
+                                    const base::FilePath& path,
+                                    Manifest::Location unused,
+                                    int unused2,
+                                    bool unused3) override {
     ADD_FAILURE() << "There should be no external extensions from files.";
     return false;
   }
 
-  virtual bool OnExternalExtensionUpdateUrlFound(
-      const std::string& id,
-      const std::string& install_parameter,
-      const GURL& update_url,
-      Manifest::Location location,
-      int unused1,
-      bool unused2) override {
+  bool OnExternalExtensionUpdateUrlFound(const std::string& id,
+                                         const std::string& install_parameter,
+                                         const GURL& update_url,
+                                         Manifest::Location location,
+                                         int unused1,
+                                         bool unused2) override {
     // Extension has the correct location.
     EXPECT_EQ(Manifest::EXTERNAL_POLICY_DOWNLOAD, location);
 
     // Provider returns the correct location when asked.
     Manifest::Location location1;
     scoped_ptr<Version> version1;
     provider_->GetExtensionDetails(id, &location1, &version1);
     EXPECT_EQ(Manifest::EXTERNAL_POLICY_DOWNLOAD, location1);
     EXPECT_FALSE(version1.get());
 
     // Remove the extension from our list.
     EXPECT_EQ(1U, expected_extensions_.erase(id));
     return true;
   }
 
-  virtual void OnExternalProviderReady(
+  void OnExternalProviderReady(
       const ExternalProviderInterface* provider) override {
     EXPECT_EQ(provider, provider_.get());
     EXPECT_TRUE(provider->IsReady());
   }
 
  private:
   std::set<std::string> expected_extensions_;
 
   scoped_ptr<TestingProfile> profile_;
 
@@ skipping 31 matching lines @@
   forced_extensions.SetString("invalid", "http://www.example.com/crx");
   forced_extensions.SetString("dddddddddddddddddddddddddddddddd",
                               std::string());
   forced_extensions.SetString("invalid", "bad");
 
   MockExternalPolicyProviderVisitor mv;
   mv.Visit(forced_extensions, expected_extensions);
 }
 
 }  // namespace extensions