base::RefCounted now DCHECKs when referenced from multiple threads.

base/deferred_sequenced_task_runner_unittest.cc

 // Copyright (c) 2013 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/deferred_sequenced_task_runner.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/memory/ref_counted.h"
 #include "base/message_loop/message_loop.h"
 #include "base/message_loop/message_loop_proxy.h"
 #include "base/threading/non_thread_safe.h"
 #include "base/threading/thread.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 class DeferredSequencedTaskRunnerTest : public testing::Test,
                                         public base::NonThreadSafe {
  public:
+  // TODO(mgiuca): Avoid using UnsafeRefCounted. http://crbug.com/469952.
   class ExecuteTaskOnDestructor :
-      public base::RefCounted<ExecuteTaskOnDestructor> {
+      public base::UnsafeRefCounted<ExecuteTaskOnDestructor> {
    public:
     ExecuteTaskOnDestructor(
         DeferredSequencedTaskRunnerTest* executor,
         int task_id)
         : executor_(executor),
           task_id_(task_id) {
     }
   private:
-    friend class base::RefCounted<ExecuteTaskOnDestructor>;
+    friend class base::UnsafeRefCounted<ExecuteTaskOnDestructor>;
     virtual ~ExecuteTaskOnDestructor() {
       executor_->ExecuteTask(task_id_);
     }
     DeferredSequencedTaskRunnerTest* executor_;
     int task_id_;
   };
 
   void ExecuteTask(int task_id) {
     base::AutoLock lock(lock_);
     executed_task_ids_.push_back(task_id);
@@ skipping 131 matching lines @@
     StartRunner();
   }
 
   // All |short_lived_object| with id |2 * i| are destroyed before the task
   // |2 * i + 1| is executed.
   EXPECT_THAT(executed_task_ids_,
               testing::ElementsAre(0, 1, 2, 3, 4, 5, 6, 7, 8, 9));
 }
 
 }  // namespace