RefCounted types should not have public destructors, base/ edition

base/threading/sequenced_worker_pool_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 <algorithm>
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 59 matching lines @@
       : lock_(),
         cond_var_(&lock_),
         started_events_(0) {
   }
 
   // Each of these tasks appends the argument to the complete sequence vector
   // so calling code can see what order they finished in.
   void FastTask(int id) {
     SignalWorkerDone(id);
   }
+
   void SlowTask(int id) {
     base::PlatformThread::Sleep(base::TimeDelta::FromSeconds(1));
     SignalWorkerDone(id);
   }
 
   void BlockTask(int id, ThreadBlocker* blocker) {
     // Note that this task has started and signal anybody waiting for that
     // to happen.
     {
       base::AutoLock lock(lock_);
@@ skipping 29 matching lines @@
     return ret;
   }
 
   void ClearCompleteSequence() {
     base::AutoLock lock(lock_);
     complete_sequence_.clear();
     started_events_ = 0;
   }
 
  private:
+  friend class base::RefCountedThreadSafe<TestTracker>;
+  ~TestTracker() {}
+
   void SignalWorkerDone(int id) {
     {
       base::AutoLock lock(lock_);
       complete_sequence_.push_back(id);
     }
     cond_var_.Signal();
   }
 
   // Protects the complete_sequence.
   base::Lock lock_;
@@ skipping 390 matching lines @@
   scoped_ptr<SequencedWorkerPoolOwner> pool_owner_;
 };
 
 INSTANTIATE_TYPED_TEST_CASE_P(
     SequencedWorkerPool, TaskRunnerTest,
     SequencedWorkerPoolTaskRunnerTestDelegate);
 
 }  // namespace
 
 }  // namespace base