Add virtual and OVERRIDE to base/ implementation files

base/metrics/stats_table_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/metrics/stats_counters.h"
 #include "base/metrics/stats_table.h"
 #include "base/shared_memory.h"
 #include "base/stringprintf.h"
 #include "base/string_piece.h"
 #include "base/test/multiprocess_test.h"
@@ skipping 59 matching lines @@
 // decremented by even threads.
 const std::string kCounterMixed = "CounterMixed";
 // The number of thread loops that we will do.
 const int kThreadLoops = 100;
 
 class StatsTableThread : public SimpleThread {
  public:
   StatsTableThread(std::string name, int id)
       : SimpleThread(name),
       id_(id) {}
-  virtual void Run();
+  virtual ~StatsTableThread() {}

[Nico, 2012/04/05 17:52:34]

Why is this needed?

[Ryan Sleevi, 2012/04/05 19:12:07]

D'oh! It isn't. This crept in when I was using an earlier (bugged) set of
changes to the Clang plugin. Thanks for catching.

+
+  virtual void Run() OVERRIDE;
+
  private:
   int id_;
 };
 
 void StatsTableThread::Run() {
   // Each thread will open the shared memory and set counters
   // concurrently in a loop.  We'll use some pauses to
   // mixup the thread scheduling.
 
   StatsCounter zero_counter(kCounterZero);
@@ skipping 316 matching lines @@
     PlatformThread::Sleep(kDuration);
   }
   EXPECT_LE(kDuration.InMilliseconds() * 2, table.GetCounterValue("t:foo"));
   EXPECT_LE(kDuration.InMilliseconds() * 2, table.GetCounterValue("t:bar"));
   EXPECT_EQ(2, table.GetCounterValue("c:bar"));
 
   DeleteShmem(kTableName);
 }
 
 }  // namespace base