TaskScheduler: Make the worker pools of TaskSchedulerImpl configurable

base/task_scheduler/task_scheduler_impl_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/task_scheduler_impl.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 20 matching lines @@
 
 struct TraitsExecutionModePair {
   TraitsExecutionModePair(const TaskTraits& traits,
                           ExecutionMode execution_mode)
       : traits(traits), execution_mode(execution_mode) {}
 
   TaskTraits traits;
   ExecutionMode execution_mode;
 };
 
-class TaskSchedulerImplTest
-    : public testing::TestWithParam<TraitsExecutionModePair> {
- protected:
-  TaskSchedulerImplTest() = default;
-
-  void SetUp() override {
-    scheduler_ = TaskSchedulerImpl::Create();
-    EXPECT_TRUE(scheduler_);
-  }
-  void TearDown() override { scheduler_->JoinForTesting(); }
-
-  std::unique_ptr<TaskSchedulerImpl> scheduler_;
-
- private:
-  DISALLOW_COPY_AND_ASSIGN(TaskSchedulerImplTest);
-};
-
 #if ENABLE_THREAD_RESTRICTIONS
 // Returns whether I/O calls are allowed on the current thread.
 bool GetIOAllowed() {
   const bool previous_value = ThreadRestrictions::SetIOAllowed(true);
   ThreadRestrictions::SetIOAllowed(previous_value);
   return previous_value;
 }
 #endif
 
 // Verify that the current thread priority and I/O restrictions are appropriate
@@ skipping 77 matching lines @@
       params.push_back(TraitsExecutionModePair(
           TaskTraits().WithPriority(priority), execution_mode));
       params.push_back(TraitsExecutionModePair(
           TaskTraits().WithPriority(priority).WithFileIO(), execution_mode));
     }
   }
 
   return params;
 }
 
+enum WorkerPoolType {
+  BACKGROUND_WORKER_POOL = 0,
+  BACKGROUND_FILE_IO_WORKER_POOL,
+  FOREGROUND_WORKER_POOL,
+  FOREGROUND_FILE_IO_WORKER_POOL,
+};
+
+size_t GetThreadPoolIndexForTraits(const TaskTraits& traits) {
+  if (traits.with_file_io()) {
+    return traits.priority() == TaskPriority::BACKGROUND
+               ? BACKGROUND_FILE_IO_WORKER_POOL
+               : FOREGROUND_FILE_IO_WORKER_POOL;
+  }
+  return traits.priority() == TaskPriority::BACKGROUND ? BACKGROUND_WORKER_POOL
+                                                       : FOREGROUND_WORKER_POOL;
+}
+
+class TaskSchedulerImplTest
+    : public testing::TestWithParam<TraitsExecutionModePair> {
+ protected:
+  TaskSchedulerImplTest() = default;
+
+  void SetUp() override {
+    using IORestriction = SchedulerWorkerPoolImpl::IORestriction;
+
+    std::vector<TaskSchedulerImpl::WorkerPoolCreationArgs> worker_pools;
+
+    EXPECT_EQ(BACKGROUND_WORKER_POOL, worker_pools.size());

[robliao, 2016/06/28 00:24:24]

These should probably be ASSERTS. If an index mismatch occurs. the test
shouldn't run.

[fdoray, 2016/06/28 15:23:58]

Done.

+    worker_pools.push_back({"TaskSchedulerBackground",
+                            ThreadPriority::BACKGROUND,
+                            IORestriction::DISALLOWED, 1U});
+
+    EXPECT_EQ(BACKGROUND_FILE_IO_WORKER_POOL, worker_pools.size());
+    worker_pools.push_back({"TaskSchedulerBackgroundFileIO",
+                            ThreadPriority::BACKGROUND, IORestriction::ALLOWED,
+                            3U});
+
+    EXPECT_EQ(FOREGROUND_WORKER_POOL, worker_pools.size());
+    worker_pools.push_back({"TaskSchedulerForeground", ThreadPriority::NORMAL,
+                            IORestriction::DISALLOWED, 4U});
+
+    DCHECK_EQ(FOREGROUND_FILE_IO_WORKER_POOL, worker_pools.size());

[robliao, 2016/06/28 00:24:24]

Typo?

[fdoray, 2016/06/28 15:23:58]

Done.

+    worker_pools.push_back({"TaskSchedulerForegroundFileIO",
+                            ThreadPriority::NORMAL, IORestriction::ALLOWED,
+                            12U});
+
+    scheduler_ = TaskSchedulerImpl::Create(worker_pools,
+                                           Bind(&GetThreadPoolIndexForTraits));
+    ASSERT_TRUE(scheduler_);
+  }
+
+  void TearDown() override { scheduler_->JoinForTesting(); }
+
+  std::unique_ptr<TaskSchedulerImpl> scheduler_;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(TaskSchedulerImplTest);
+};
+
 }  // namespace
 
 // Verifies that a Task posted via PostTaskWithTraits with parameterized
 // TaskTraits runs on a thread with the expected priority and I/O restrictions.
 // The ExecutionMode parameter is ignored by this test.
 TEST_P(TaskSchedulerImplTest, PostTaskWithTraits) {
   WaitableEvent task_ran(WaitableEvent::ResetPolicy::MANUAL,
                          WaitableEvent::InitialState::NOT_SIGNALED);
   scheduler_->PostTaskWithTraits(
       FROM_HERE, GetParam().traits,
@@ skipping 22 matching lines @@
 }
 
 INSTANTIATE_TEST_CASE_P(OneTraitsExecutionModePair,
                         TaskSchedulerImplTest,
                         ::testing::ValuesIn(GetTraitsExecutionModePairs()));
 
 // Spawns threads that simultaneously post Tasks to TaskRunners with various
 // TaskTraits and ExecutionModes. Verifies that each Task runs on a thread with
 // the expected priority and I/O restrictions and respects the characteristics
 // of its ExecutionMode.
-TEST(TaskSchedulerImplTest, MultipleTraitsExecutionModePairs) {
-  std::unique_ptr<TaskSchedulerImpl> scheduler = TaskSchedulerImpl::Create();
-
+TEST_F(TaskSchedulerImplTest, MultipleTraitsExecutionModePairs) {
   std::vector<std::unique_ptr<ThreadPostingTasks>> threads_posting_tasks;
   for (const auto& traits_execution_mode_pair : GetTraitsExecutionModePairs()) {
     threads_posting_tasks.push_back(WrapUnique(new ThreadPostingTasks(
-        scheduler.get(), traits_execution_mode_pair.traits,
+        scheduler_.get(), traits_execution_mode_pair.traits,
         traits_execution_mode_pair.execution_mode)));
     threads_posting_tasks.back()->Start();
   }
 
   for (const auto& thread : threads_posting_tasks) {
     thread->WaitForAllTasksToRun();
     thread->Join();
   }
-
-  scheduler->JoinForTesting();
 }
 
 // TODO(fdoray): Add tests with Sequences that move around worker pools once
 // child TaskRunners are supported.
 
 }  // namespace internal
 }  // namespace base