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

Unified Diff: base/task_scheduler/scheduler_worker_pool_impl.h

Issue 2862143004: Remove SchedulerWorkerPoolImpl::ReEnqueueSequenceCallback. (Closed)
Patch Set: CR-robliao-remove-includes Created 3 years, 7 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
« no previous file with comments | « base/task_scheduler/scheduler_worker_pool.h ('k') | base/task_scheduler/scheduler_worker_pool_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/task_scheduler/scheduler_worker_pool_impl.h
diff --git a/base/task_scheduler/scheduler_worker_pool_impl.h b/base/task_scheduler/scheduler_worker_pool_impl.h
index 6fc5ee0569132627c439e343adaa06e685ff4c07..b61ed0910e33ce8ed3ed1e8f22379b00604e6aa6 100644
--- a/base/task_scheduler/scheduler_worker_pool_impl.h
+++ b/base/task_scheduler/scheduler_worker_pool_impl.h
@@ -12,7 +12,6 @@
#include <vector>
#include "base/base_export.h"
-#include "base/callback.h"
#include "base/logging.h"
#include "base/macros.h"
#include "base/memory/ref_counted.h"
@@ -46,23 +45,17 @@ class TaskTracker;
// This class is thread-safe.
class BASE_EXPORT SchedulerWorkerPoolImpl : public SchedulerWorkerPool {
public:
- // Callback invoked when a Sequence isn't empty after a worker pops a Task
- // from it.
- using ReEnqueueSequenceCallback = Callback<void(scoped_refptr<Sequence>)>;
-
// Constructs a pool without workers.
//
// |name| is used to label the pool's threads ("TaskScheduler" + |name| +
// index) and histograms ("TaskScheduler." + histogram name + "." + |name| +
// extra suffixes). |priority_hint| is the preferred thread priority; the
// actual thread priority depends on shutdown state and platform capabilities.
- // |re_enqueue_sequence_callback| is invoked when a Sequence isn't empty after
- // a worker pops a Task from it. |task_tracker| keeps track of tasks.
- // |delayed_task_manager| handles tasks posted with a delay.
+ // |task_tracker| keeps track of tasks. |delayed_task_manager| handles tasks
+ // posted with a delay.
SchedulerWorkerPoolImpl(
const std::string& name,
ThreadPriority priority_hint,
- ReEnqueueSequenceCallback re_enqueue_sequence_callback,
TaskTracker* task_tracker,
DelayedTaskManager* delayed_task_manager);
@@ -80,8 +73,6 @@ class BASE_EXPORT SchedulerWorkerPoolImpl : public SchedulerWorkerPool {
const TaskTraits& traits) override;
scoped_refptr<SequencedTaskRunner> CreateSequencedTaskRunnerWithTraits(
const TaskTraits& traits) override;
- void ReEnqueueSequence(scoped_refptr<Sequence> sequence,
- const SequenceSortKey& sequence_sort_key) override;
bool PostTaskWithSequence(std::unique_ptr<Task> task,
scoped_refptr<Sequence> sequence) override;
void PostTaskWithSequenceNow(std::unique_ptr<Task> task,
@@ -143,7 +134,6 @@ class BASE_EXPORT SchedulerWorkerPoolImpl : public SchedulerWorkerPool {
const std::string name_;
const ThreadPriority priority_hint_;
- const ReEnqueueSequenceCallback re_enqueue_sequence_callback_;
// PriorityQueue from which all threads of this worker pool get work.
PriorityQueue shared_priority_queue_;
« no previous file with comments | « base/task_scheduler/scheduler_worker_pool.h ('k') | base/task_scheduler/scheduler_worker_pool_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698