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

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, 4 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 497db9c06430259a5c28ce3dc2126f68d9c08068..6943a58a60c43853a4d40d15153f619e6c9f0c95 100644
--- a/base/task_scheduler/scheduler_worker_pool_impl.cc
+++ b/base/task_scheduler/scheduler_worker_pool_impl.cc
@@ -59,7 +59,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);
}
@@ -595,9 +595,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)

Powered by Google App Engine
This is Rietveld 408576698