Standardize usage of virtual/override/final specifiers in base/.

base/files/file_path_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 "base/basictypes.h"
 #include "base/files/file_path.h"
 #include "base/strings/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 
@@ skipping 32 matching lines @@
 
 struct UTF8TestData {
   const FilePath::CharType* native;
   const char* utf8;
 };
 
 // file_util winds up using autoreleased objects on the Mac, so this needs
 // to be a PlatformTest
 class FilePathTest : public PlatformTest {
  protected:
-  virtual void SetUp() override {
-    PlatformTest::SetUp();
-  }
-  virtual void TearDown() override {
-    PlatformTest::TearDown();
-  }
+  void SetUp() override { PlatformTest::SetUp(); }

[Nico, 2014/12/22 22:05:42]

This doesn't have any effect. Can FilePathTest just be removed?

+  void TearDown() override { PlatformTest::TearDown(); }
 };
 
 TEST_F(FilePathTest, DirName) {
   const struct UnaryTestData cases[] = {
     { FPL(""),              FPL(".") },
     { FPL("aa"),            FPL(".") },
     { FPL("/aa/bb"),        FPL("/aa") },
     { FPL("/aa/bb/"),       FPL("/aa") },
     { FPL("/aa/bb//"),      FPL("/aa") },
     { FPL("/aa/bb/ccc"),    FPL("/aa/bb") },
@@ skipping 1206 matching lines @@
   for (size_t i = 0; i < arraysize(cases); ++i) {
     FilePath input(cases[i].input);
     bool observed = input.IsContentUri();
     EXPECT_EQ(cases[i].expected, observed) <<
               "i: " << i << ", input: " << input.value();
   }
 }
 #endif
 
 }  // namespace base