Add virtual and OVERRIDE to base/ implementation files

base/threading/thread_local_unittest.cc

 // Copyright (c) 2010 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/logging.h"
 #include "base/threading/simple_thread.h"
 #include "base/threading/thread_local.h"
 #include "base/synchronization/waitable_event.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace base {
 
 namespace {
 
 class ThreadLocalTesterBase : public base::DelegateSimpleThreadPool::Delegate {
  public:
   typedef base::ThreadLocalPointer<ThreadLocalTesterBase> TLPType;
 
   ThreadLocalTesterBase(TLPType* tlp, base::WaitableEvent* done)
       : tlp_(tlp), done_(done) { }

[jar (doing other things), 2012/04/05 21:11:30]

nit: one initializer per line.

-  ~ThreadLocalTesterBase() { }
+  virtual ~ThreadLocalTesterBase() { }
 
  protected:
   TLPType* tlp_;
   base::WaitableEvent* done_;
 };
 
 class SetThreadLocal : public ThreadLocalTesterBase {
  public:
   SetThreadLocal(TLPType* tlp, base::WaitableEvent* done)
       : ThreadLocalTesterBase(tlp, done), val_(NULL) { }

[jar (doing other things), 2012/04/05 21:11:30]

nit: one init per line.

-  ~SetThreadLocal() { }
+  virtual ~SetThreadLocal() { }
 
   void set_value(ThreadLocalTesterBase* val) { val_ = val; }
 
-  virtual void Run() {
+  virtual void Run() OVERRIDE {
     DCHECK(!done_->IsSignaled());
     tlp_->Set(val_);
     done_->Signal();
   }
 
  private:
   ThreadLocalTesterBase* val_;
 };
 
 class GetThreadLocal : public ThreadLocalTesterBase {
  public:
   GetThreadLocal(TLPType* tlp, base::WaitableEvent* done)
       : ThreadLocalTesterBase(tlp, done), ptr_(NULL) { }
-  ~GetThreadLocal() { }
+  virtual ~GetThreadLocal() { }
 
   void set_ptr(ThreadLocalTesterBase** ptr) { ptr_ = ptr; }
 
-  virtual void Run() {
+  virtual void Run() OVERRIDE {
     DCHECK(!done_->IsSignaled());
     *ptr_ = tlp_->Get();
     done_->Signal();
   }
 
  private:
   ThreadLocalTesterBase** ptr_;
 };
 
 }  // namespace
@@ skipping 89 matching lines @@
   }
 
   // Our slot should have been freed, we're all reset.
   {
     base::ThreadLocalBoolean tlb;
     EXPECT_FALSE(tlb.Get());
   }
 }
 
 }  // namespace base