Chromium Code Reviews| Index: base/task_scheduler/scheduler_worker_stack_unittest.cc |
| diff --git a/base/task_scheduler/scheduler_worker_thread_stack_unittest.cc b/base/task_scheduler/scheduler_worker_stack_unittest.cc |
| similarity index 73% |
| rename from base/task_scheduler/scheduler_worker_thread_stack_unittest.cc |
| rename to base/task_scheduler/scheduler_worker_stack_unittest.cc |
| index 629b77952249c5cbb5bde78cd788e2bc18c2f877..f0eb76a3c74d12862038057a19812658bf1cf086 100644 |
| --- a/base/task_scheduler/scheduler_worker_thread_stack_unittest.cc |
| +++ b/base/task_scheduler/scheduler_worker_stack_unittest.cc |
| @@ -2,12 +2,12 @@ |
| // 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/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_thread.h" |
| +#include "base/task_scheduler/scheduler_worker.h" |
| #include "base/task_scheduler/sequence.h" |
| #include "base/task_scheduler/task_tracker.h" |
| #include "base/task_scheduler/test_utils.h" |
| @@ -20,12 +20,10 @@ namespace internal { |
| namespace { |
| -class MockSchedulerWorkerThreadDelegate |
| - : public SchedulerWorkerThread::Delegate { |
| +class MockSchedulerWorkerDelegate : public SchedulerWorker::Delegate { |
| public: |
| - void OnMainEntry(SchedulerWorkerThread* worker_thread) override {} |
| - scoped_refptr<Sequence> GetWork( |
| - SchedulerWorkerThread* worker_thread) override { |
| + void OnMainEntry(SchedulerWorker* worker) override {} |
| + scoped_refptr<Sequence> GetWork(SchedulerWorker* worker) override { |
| return nullptr; |
| } |
| void ReEnqueueSequence(scoped_refptr<Sequence> sequence) override { |
| @@ -38,20 +36,20 @@ class MockSchedulerWorkerThreadDelegate |
| } |
| }; |
| -class TaskSchedulerWorkerThreadStackTest : public testing::Test { |
| +class TaskSchedulerWorkerStackTest : public testing::Test { |
| protected: |
| void SetUp() override { |
| - thread_a_ = SchedulerWorkerThread::Create( |
| + thread_a_ = SchedulerWorker::Create( |
|
fdoray
2016/06/20 15:08:40
worker_a_
robliao
2016/06/20 17:50:09
Done.
|
| ThreadPriority::NORMAL, |
| - WrapUnique(new MockSchedulerWorkerThreadDelegate), &task_tracker_); |
| + WrapUnique(new MockSchedulerWorkerDelegate), &task_tracker_); |
| ASSERT_TRUE(thread_a_); |
| - thread_b_ = SchedulerWorkerThread::Create( |
| + thread_b_ = SchedulerWorker::Create( |
|
fdoray
2016/06/20 15:08:40
worker_b_
robliao
2016/06/20 17:50:09
Done.
|
| ThreadPriority::NORMAL, |
| - WrapUnique(new MockSchedulerWorkerThreadDelegate), &task_tracker_); |
| + WrapUnique(new MockSchedulerWorkerDelegate), &task_tracker_); |
| ASSERT_TRUE(thread_b_); |
| - thread_c_ = SchedulerWorkerThread::Create( |
| + thread_c_ = SchedulerWorker::Create( |
|
fdoray
2016/06/20 15:08:40
worker_c_
robliao
2016/06/20 17:50:09
Done.
|
| ThreadPriority::NORMAL, |
| - WrapUnique(new MockSchedulerWorkerThreadDelegate), &task_tracker_); |
| + WrapUnique(new MockSchedulerWorkerDelegate), &task_tracker_); |
| ASSERT_TRUE(thread_c_); |
| } |
| @@ -61,9 +59,9 @@ class TaskSchedulerWorkerThreadStackTest : public testing::Test { |
| thread_c_->JoinForTesting(); |
| } |
| - std::unique_ptr<SchedulerWorkerThread> thread_a_; |
| - std::unique_ptr<SchedulerWorkerThread> thread_b_; |
| - std::unique_ptr<SchedulerWorkerThread> thread_c_; |
| + std::unique_ptr<SchedulerWorker> thread_a_; |
| + std::unique_ptr<SchedulerWorker> thread_b_; |
| + std::unique_ptr<SchedulerWorker> thread_c_; |
| private: |
| TaskTracker task_tracker_; |
| @@ -72,8 +70,8 @@ class TaskSchedulerWorkerThreadStackTest : public testing::Test { |
| } // namespace |
| // Verify that Push() and Pop() add/remove values in FIFO order. |
| -TEST_F(TaskSchedulerWorkerThreadStackTest, PushPop) { |
| - SchedulerWorkerThreadStack stack; |
| +TEST_F(TaskSchedulerWorkerStackTest, PushPop) { |
| + SchedulerWorkerStack stack; |
| EXPECT_TRUE(stack.IsEmpty()); |
| EXPECT_EQ(0U, stack.Size()); |
| @@ -111,8 +109,8 @@ TEST_F(TaskSchedulerWorkerThreadStackTest, PushPop) { |
| } |
| // Verify that a value can be removed by Remove(). |
| -TEST_F(TaskSchedulerWorkerThreadStackTest, Remove) { |
| - SchedulerWorkerThreadStack stack; |
| +TEST_F(TaskSchedulerWorkerStackTest, Remove) { |
| + SchedulerWorkerStack stack; |
| EXPECT_TRUE(stack.IsEmpty()); |
| EXPECT_EQ(0U, stack.Size()); |
| @@ -142,8 +140,8 @@ TEST_F(TaskSchedulerWorkerThreadStackTest, Remove) { |
| } |
| // Verify that a value can be pushed again after it has been removed. |
| -TEST_F(TaskSchedulerWorkerThreadStackTest, PushAfterRemove) { |
| - SchedulerWorkerThreadStack stack; |
| +TEST_F(TaskSchedulerWorkerStackTest, PushAfterRemove) { |
| + SchedulerWorkerStack stack; |
| EXPECT_EQ(0U, stack.Size()); |
| EXPECT_TRUE(stack.IsEmpty()); |
| @@ -161,8 +159,8 @@ TEST_F(TaskSchedulerWorkerThreadStackTest, PushAfterRemove) { |
| } |
| // Verify that Push() DCHECKs when a value is inserted twice. |
| -TEST_F(TaskSchedulerWorkerThreadStackTest, PushTwice) { |
| - SchedulerWorkerThreadStack stack; |
| +TEST_F(TaskSchedulerWorkerStackTest, PushTwice) { |
| + SchedulerWorkerStack stack; |
| stack.Push(thread_a_.get()); |
| EXPECT_DCHECK_DEATH({ stack.Push(thread_a_.get()); }, ""); |
| } |