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

Unified Diff: base/task_scheduler/scheduler_worker_thread.cc

Issue 1903103007: TaskScheduler: Make SchedulerWorkerThread own its delegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@sched_2b_remove_utils
Patch Set: rebase Created 4 years, 8 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_thread.cc
diff --git a/base/task_scheduler/scheduler_worker_thread.cc b/base/task_scheduler/scheduler_worker_thread.cc
index b571ebca82e7347088db4694de39778f7f38c7c0..11c88af415471a3b2aad62232c755dfcb46856cd 100644
--- a/base/task_scheduler/scheduler_worker_thread.cc
+++ b/base/task_scheduler/scheduler_worker_thread.cc
@@ -14,13 +14,13 @@
namespace base {
namespace internal {
-std::unique_ptr<SchedulerWorkerThread>
-SchedulerWorkerThread::CreateSchedulerWorkerThread(
+std::unique_ptr<SchedulerWorkerThread> SchedulerWorkerThread::Create(
ThreadPriority thread_priority,
- Delegate* delegate,
+ std::unique_ptr<Delegate> delegate,
TaskTracker* task_tracker) {
std::unique_ptr<SchedulerWorkerThread> worker_thread(
- new SchedulerWorkerThread(thread_priority, delegate, task_tracker));
+ new SchedulerWorkerThread(thread_priority, std::move(delegate),
+ task_tracker));
if (worker_thread->thread_handle_.is_null())
return nullptr;
@@ -45,10 +45,10 @@ void SchedulerWorkerThread::JoinForTesting() {
}
SchedulerWorkerThread::SchedulerWorkerThread(ThreadPriority thread_priority,
- Delegate* delegate,
+ std::unique_ptr<Delegate> delegate,
TaskTracker* task_tracker)
: wake_up_event_(false, false),
- delegate_(delegate),
+ delegate_(std::move(delegate)),
task_tracker_(task_tracker) {
DCHECK(delegate_);
DCHECK(task_tracker_);
« no previous file with comments | « base/task_scheduler/scheduler_worker_thread.h ('k') | base/task_scheduler/scheduler_worker_thread_stack_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698