Linux/ChromeOS Chromium style checker cleanup, chrome/browser edition.

chrome/browser/net/chrome_net_log_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 "chrome/browser/net/chrome_net_log.h"
 
 #include "base/synchronization/waitable_event.h"
 #include "base/threading/simple_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 const int kThreads = 10;
 const int kEvents = 100;
 
 class CountingObserver : public net::NetLog::ThreadSafeObserver {
  public:
   CountingObserver() : count_(0) {}
 
-  ~CountingObserver() {
+  virtual ~CountingObserver() {
     if (net_log())
       net_log()->RemoveThreadSafeObserver(this);
   }
 
   virtual void OnAddEntry(const net::NetLog::Entry& entry) OVERRIDE {
     ++count_;
   }
 
   int count() const { return count_; }
 
@@ skipping 38 matching lines @@
   // each thread completes before the next one starts.
   base::WaitableEvent* start_event_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeNetLogTestThread);
 };
 
 // A thread that adds a bunch of events to the NetLog.
 class AddEventsTestThread : public ChromeNetLogTestThread {
  public:
   AddEventsTestThread() {}
-  ~AddEventsTestThread() {}
+  virtual ~AddEventsTestThread() {}
 
  private:
   virtual void RunTestThread() OVERRIDE {
     for (int i = 0; i < kEvents; ++i)
       AddEvent(net_log_);
   }
 
   DISALLOW_COPY_AND_ASSIGN(AddEventsTestThread);
 };
 
 // A thread that adds and removes an observer from the NetLog repeatedly.
 class AddRemoveObserverTestThread : public ChromeNetLogTestThread {
  public:
   AddRemoveObserverTestThread() {}
 
-  ~AddRemoveObserverTestThread() {
+  virtual ~AddRemoveObserverTestThread() {
     EXPECT_TRUE(!observer_.net_log());
   }
 
  private:
   virtual void RunTestThread() OVERRIDE {
     for (int i = 0; i < kEvents; ++i) {
       ASSERT_FALSE(observer_.net_log());
 
       net_log_->AddThreadSafeObserver(&observer_, net::NetLog::LOG_BASIC);
       ASSERT_EQ(net_log_, observer_.net_log());
@@ skipping 156 matching lines @@
 // threads works.
 TEST(ChromeNetLogTest, NetLogAddRemoveObserverThreads) {
   ChromeNetLog net_log;
 
   // Run a bunch of threads to completion, each of which will repeatedly add
   // and remove an observer, and set its logging level.
   RunTestThreads<AddRemoveObserverTestThread>(&net_log);
 }
 
 }  // namespace