Separate the create and start phases in SchedulerSingleThreadTaskRunnerManager.

base/task_scheduler/scheduler_worker_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_stack.h"
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/task_scheduler/scheduler_worker.h"
@@ skipping 26 matching lines @@
   }
   bool CanDetach(SchedulerWorker* worker) override {
     return false;
   }
   void OnDetach() override { ADD_FAILURE() << "Unexpected call to OnDetach()"; }
 };
 
 class TaskSchedulerWorkerStackTest : public testing::Test {
  protected:
   void SetUp() override {
-    worker_a_ = SchedulerWorker::Create(
-        ThreadPriority::NORMAL,
-        WrapUnique(new MockSchedulerWorkerDelegate), &task_tracker_,
-        SchedulerWorker::InitialState::ALIVE);
+    worker_a_ = make_scoped_refptr(new SchedulerWorker(
+        ThreadPriority::NORMAL, WrapUnique(new MockSchedulerWorkerDelegate),
+        &task_tracker_));
     ASSERT_TRUE(worker_a_);
-    worker_b_ = SchedulerWorker::Create(
-        ThreadPriority::NORMAL,
-        WrapUnique(new MockSchedulerWorkerDelegate), &task_tracker_,
-        SchedulerWorker::InitialState::ALIVE);
+    worker_b_ = make_scoped_refptr(new SchedulerWorker(
+        ThreadPriority::NORMAL, WrapUnique(new MockSchedulerWorkerDelegate),
+        &task_tracker_));
     ASSERT_TRUE(worker_b_);
-    worker_c_ = SchedulerWorker::Create(
-        ThreadPriority::NORMAL,
-        WrapUnique(new MockSchedulerWorkerDelegate), &task_tracker_,
-        SchedulerWorker::InitialState::ALIVE);
+    worker_c_ = make_scoped_refptr(new SchedulerWorker(
+        ThreadPriority::NORMAL, WrapUnique(new MockSchedulerWorkerDelegate),
+        &task_tracker_));
     ASSERT_TRUE(worker_c_);
   }
 
-  void TearDown() override {
-    worker_a_->JoinForTesting();
-    worker_b_->JoinForTesting();
-    worker_c_->JoinForTesting();
-  }
-
   scoped_refptr<SchedulerWorker> worker_a_;
   scoped_refptr<SchedulerWorker> worker_b_;
   scoped_refptr<SchedulerWorker> worker_c_;
 
  private:
   TaskTracker task_tracker_;
 };
 
 }  // namespace
 
@@ skipping 170 matching lines @@
 
 // Verify that Push() DCHECKs when a value is inserted twice.
 TEST_F(TaskSchedulerWorkerStackTest, PushTwice) {
   SchedulerWorkerStack stack;
   stack.Push(worker_a_.get());
   EXPECT_DCHECK_DEATH({ stack.Push(worker_a_.get()); });
 }
 
 }  // namespace internal
 }  // namespace base