Standardize usage of virtual/override/final in extensions/

extensions/common/manifest_handler_unittest.cc

 // Copyright (c) 2013 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 <vector>
 
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/stl_util.h"
@@ skipping 65 matching lines @@
 
   class TestManifestHandler : public ManifestHandler {
    public:
     TestManifestHandler(const std::string& name,
                         const std::vector<std::string>& keys,
                         const std::vector<std::string>& prereqs,
                         ParsingWatcher* watcher)
         : name_(name), keys_(keys), prereqs_(prereqs), watcher_(watcher) {
     }
 
-    virtual bool Parse(Extension* extension, base::string16* error) override {
+    bool Parse(Extension* extension, base::string16* error) override {
       watcher_->Record(name_);
       return true;
     }
 
-    virtual const std::vector<std::string> PrerequisiteKeys() const override {
+    const std::vector<std::string> PrerequisiteKeys() const override {
       return prereqs_;
     }
 
    protected:
     std::string name_;
     std::vector<std::string> keys_;
     std::vector<std::string> prereqs_;
     ParsingWatcher* watcher_;
 
-    virtual const std::vector<std::string> Keys() const override {
-      return keys_;
-    }
+    const std::vector<std::string> Keys() const override { return keys_; }
   };
 
   class FailingTestManifestHandler : public TestManifestHandler {
    public:
     FailingTestManifestHandler(const std::string& name,
                                const std::vector<std::string>& keys,
                                const std::vector<std::string>& prereqs,
                                ParsingWatcher* watcher)
         : TestManifestHandler(name, keys, prereqs, watcher) {
     }
-    virtual bool Parse(Extension* extension, base::string16* error) override {
+    bool Parse(Extension* extension, base::string16* error) override {
       *error = base::ASCIIToUTF16(name_);
       return false;
     }
   };
 
   class AlwaysParseTestManifestHandler : public TestManifestHandler {
    public:
     AlwaysParseTestManifestHandler(const std::string& name,
                                    const std::vector<std::string>& keys,
                                    const std::vector<std::string>& prereqs,
                                    ParsingWatcher* watcher)
         : TestManifestHandler(name, keys, prereqs, watcher) {
     }
 
-    virtual bool AlwaysParseForType(Manifest::Type type) const override {
-      return true;
-    }
+    bool AlwaysParseForType(Manifest::Type type) const override { return true; }
   };
 
   class TestManifestValidator : public ManifestHandler {
    public:
     TestManifestValidator(bool return_value,
                           bool always_validate,
                           std::vector<std::string> keys)
         : return_value_(return_value),
           always_validate_(always_validate),
           keys_(keys) {
     }
 
-    virtual bool Parse(Extension* extension, base::string16* error) override {
+    bool Parse(Extension* extension, base::string16* error) override {
       return true;
     }
 
-    virtual bool Validate(
-        const Extension* extension,
-        std::string* error,
-        std::vector<InstallWarning>* warnings) const override {
+    bool Validate(const Extension* extension,
+                  std::string* error,
+                  std::vector<InstallWarning>* warnings) const override {
       return return_value_;
     }
 
-    virtual bool AlwaysValidateForType(Manifest::Type type) const override {
+    bool AlwaysValidateForType(Manifest::Type type) const override {
       return always_validate_;
     }
 
    private:
-    virtual const std::vector<std::string> Keys() const override {
-      return keys_;
-    }
+    const std::vector<std::string> Keys() const override { return keys_; }
 
    protected:
     bool return_value_;
     bool always_validate_;
     std::vector<std::string> keys_;
   };
 };
 
 TEST_F(ManifestHandlerTest, DependentHandlers) {
   ScopedTestingManifestHandlerRegistry registry;
@@ skipping 99 matching lines @@
   EXPECT_TRUE(
       ManifestHandler::ValidateExtension(extension.get(), &error, &warnings));
 
   // Validates "a" and fails.
   (new TestManifestValidator(false, true, SingleKey("a")))->Register();
   EXPECT_FALSE(
       ManifestHandler::ValidateExtension(extension.get(), &error, &warnings));
 }
 
 }  // namespace extensions