Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3441)

Unified Diff: base/task_scheduler/scheduler_worker_pool_impl.cc

Issue 2258713003: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/task_scheduler/scheduler_worker_pool_impl.cc
diff --git a/base/task_scheduler/scheduler_worker_pool_impl.cc b/base/task_scheduler/scheduler_worker_pool_impl.cc
index 3774c47f082d85128df2be41fef4d0d600e00696..e5599dd6e6e3ae010ba04998aadf2ec9fbe518d0 100644
--- a/base/task_scheduler/scheduler_worker_pool_impl.cc
+++ b/base/task_scheduler/scheduler_worker_pool_impl.cc
@@ -61,7 +61,7 @@ class SchedulerParallelTaskRunner : public TaskRunner {
TimeDelta delay) override {
// Post the task as part of a one-off single-task Sequence.
return worker_pool_->PostTaskWithSequence(
- WrapUnique(new Task(from_here, closure, traits_, delay)),
+ MakeUnique<Task>(from_here, closure, traits_, delay),
make_scoped_refptr(new Sequence), nullptr);
}
@@ -643,9 +643,9 @@ bool SchedulerWorkerPoolImpl::Initialize(
for (size_t i = 0; i < max_threads; ++i) {
std::unique_ptr<SchedulerWorker> worker = SchedulerWorker::Create(
- priority_hint, WrapUnique(new SchedulerWorkerDelegateImpl(
+ priority_hint, MakeUnique<SchedulerWorkerDelegateImpl>(
this, re_enqueue_sequence_callback,
- &shared_priority_queue_, static_cast<int>(i))),
+ &shared_priority_queue_, static_cast<int>(i)),
task_tracker_, i == 0 ? SchedulerWorker::InitialState::ALIVE
: SchedulerWorker::InitialState::DETACHED);
if (!worker)
« no previous file with comments | « base/task_scheduler/scheduler_service_thread.cc ('k') | base/task_scheduler/scheduler_worker_pool_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698