TaskScheduler: Make SchedulerWorkerThread own its delegate.

base/task_scheduler/scheduler_worker_thread_stack_unittest.cc

 // Copyright 2016 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/task_scheduler/scheduler_worker_thread_stack.h"
 
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/task_scheduler/scheduler_worker_thread.h"
 #include "base/task_scheduler/sequence.h"
 #include "base/task_scheduler/task_tracker.h"
 #include "base/task_scheduler/test_utils.h"
 #include "base/threading/platform_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace base {
 namespace internal {
 
 namespace {
 
 class MockSchedulerWorkerThreadDelegate
     : public SchedulerWorkerThread::Delegate {
  public:
   void OnMainEntry() override {}
   scoped_refptr<Sequence> GetWork(
       SchedulerWorkerThread* worker_thread) override {
     return nullptr;
   }
   void ReEnqueueSequence(scoped_refptr<Sequence> sequence) override {
     NOTREACHED();
   }
 };
 
 class TaskSchedulerWorkerThreadStackTest : public testing::Test {
  protected:
   void SetUp() override {
-    thread_a_ = SchedulerWorkerThread::CreateSchedulerWorkerThread(
-        ThreadPriority::NORMAL, &delegate_, &task_tracker_);
+    thread_a_ = SchedulerWorkerThread::Create(
+        ThreadPriority::NORMAL,
+        WrapUnique(new MockSchedulerWorkerThreadDelegate), &task_tracker_);
     ASSERT_TRUE(thread_a_);
-    thread_b_ = SchedulerWorkerThread::CreateSchedulerWorkerThread(
-        ThreadPriority::NORMAL, &delegate_, &task_tracker_);
+    thread_b_ = SchedulerWorkerThread::Create(
+        ThreadPriority::NORMAL,
+        WrapUnique(new MockSchedulerWorkerThreadDelegate), &task_tracker_);
     ASSERT_TRUE(thread_b_);
-    thread_c_ = SchedulerWorkerThread::CreateSchedulerWorkerThread(
-        ThreadPriority::NORMAL, &delegate_, &task_tracker_);
+    thread_c_ = SchedulerWorkerThread::Create(
+        ThreadPriority::NORMAL,
+        WrapUnique(new MockSchedulerWorkerThreadDelegate), &task_tracker_);
     ASSERT_TRUE(thread_c_);
   }
 
   void TearDown() override {
     thread_a_->JoinForTesting();
     thread_b_->JoinForTesting();
     thread_c_->JoinForTesting();
   }
 
   std::unique_ptr<SchedulerWorkerThread> thread_a_;
   std::unique_ptr<SchedulerWorkerThread> thread_b_;
   std::unique_ptr<SchedulerWorkerThread> thread_c_;
 
  private:
-  MockSchedulerWorkerThreadDelegate delegate_;
   TaskTracker task_tracker_;
 };
 
 }  // namespace
 
 // Verify that Push() and Pop() add/remove values in FIFO order.
 TEST_F(TaskSchedulerWorkerThreadStackTest, PushPop) {
   SchedulerWorkerThreadStack stack;
   EXPECT_TRUE(stack.IsEmpty());
   EXPECT_EQ(0U, stack.Size());
@@ skipping 83 matching lines @@
 
 // Verify that Push() DCHECKs when a value is inserted twice.
 TEST_F(TaskSchedulerWorkerThreadStackTest, PushTwice) {
   SchedulerWorkerThreadStack stack;
   stack.Push(thread_a_.get());
   EXPECT_DCHECK_DEATH({ stack.Push(thread_a_.get()); }, "");
 }
 
 }  // namespace internal
 }  // namespace base