Refactor WorkerPoolCreationArgs to a Read-Only WorkerPoolParams

base/task_scheduler/task_scheduler_impl.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 <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/memory/ptr_util.h"
 #include "base/task_scheduler/scheduler_service_thread.h"
 #include "base/task_scheduler/sequence_sort_key.h"
 #include "base/task_scheduler/task.h"
+#include "base/task_scheduler/worker_pool_params.h"
 #include "base/time/time.h"
 
 namespace base {
 namespace internal {
 
 // static
 std::unique_ptr<TaskSchedulerImpl> TaskSchedulerImpl::Create(
-    const std::vector<WorkerPoolCreationArgs>& worker_pools,
+    const std::vector<WorkerPoolParams>& worker_pools,
     const WorkerPoolIndexForTraitsCallback&
         worker_pool_index_for_traits_callback) {
   std::unique_ptr<TaskSchedulerImpl> scheduler(
       new TaskSchedulerImpl(worker_pool_index_for_traits_callback));
   scheduler->Initialize(worker_pools);
   return scheduler;
 }
 
 TaskSchedulerImpl::~TaskSchedulerImpl() {
 #if DCHECK_IS_ON()
@@ skipping 44 matching lines @@
 #if DCHECK_IS_ON()
       ,
       join_for_testing_returned_(WaitableEvent::ResetPolicy::MANUAL,
                                  WaitableEvent::InitialState::NOT_SIGNALED)
 #endif
 {
   DCHECK(!worker_pool_index_for_traits_callback_.is_null());
 }
 
 void TaskSchedulerImpl::Initialize(
-    const std::vector<WorkerPoolCreationArgs>& worker_pools) {
+    const std::vector<WorkerPoolParams>& worker_pools) {
   DCHECK(!worker_pools.empty());
 
   const SchedulerWorkerPoolImpl::ReEnqueueSequenceCallback
       re_enqueue_sequence_callback =
           Bind(&TaskSchedulerImpl::ReEnqueueSequenceCallback, Unretained(this));
 
   for (const auto& worker_pool : worker_pools) {
     // Passing pointers to objects owned by |this| to
     // SchedulerWorkerPoolImpl::Create() is safe because a TaskSchedulerImpl
     // can't be deleted before all its worker pools have been joined.
     worker_pools_.push_back(SchedulerWorkerPoolImpl::Create(
-        worker_pool.name, worker_pool.thread_priority, worker_pool.max_threads,
-        worker_pool.io_restriction, re_enqueue_sequence_callback,
-        &task_tracker_, &delayed_task_manager_));
+        worker_pool, re_enqueue_sequence_callback, &task_tracker_,
+        &delayed_task_manager_));
     CHECK(worker_pools_.back());
   }
 
   service_thread_ = SchedulerServiceThread::Create(&task_tracker_,
                                                    &delayed_task_manager_);
   CHECK(service_thread_);
 }
 
 SchedulerWorkerPool* TaskSchedulerImpl::GetWorkerPoolForTraits(
     const TaskTraits& traits) {
@@ skipping 17 matching lines @@
   GetWorkerPoolForTraits(traits)->ReEnqueueSequence(std::move(sequence),
                                                     sort_key);
 }
 
 void TaskSchedulerImpl::OnDelayedRunTimeUpdated() {
   service_thread_->WakeUp();
 }
 
 }  // namespace internal
 }  // namespace base