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

Unified Diff: third_party/WebKit/Source/platform/scheduler/base/work_queue_sets.cc

Issue 2258713004: Make tasks cancellable inside the blink scheduler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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: third_party/WebKit/Source/platform/scheduler/base/work_queue_sets.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets.cc b/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets.cc
index 47ffb0e8877e176d302872447602ef40de86b246..f11d3457cbcc3c3b2e1a85db5a0273430e1ccfe0 100644
--- a/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets.cc
+++ b/third_party/WebKit/Source/platform/scheduler/base/work_queue_sets.cc
@@ -75,6 +75,7 @@ void WorkQueueSets::OnPushQueue(WorkQueue* work_queue) {
}
void WorkQueueSets::OnPopQueue(WorkQueue* work_queue) {
+ // Assume that |work_queue| contains the lowest enqueue_order.
size_t set_index = work_queue->work_queue_set_index();
DCHECK_EQ(this, work_queue->work_queue_sets());
DCHECK_LT(set_index, enqueue_order_to_work_queue_maps_.size());
@@ -94,6 +95,26 @@ void WorkQueueSets::OnPopQueue(WorkQueue* work_queue) {
std::make_pair(enqueue_order, work_queue));
}
+void WorkQueueSets::OnQueueHeadChanged(WorkQueue* work_queue,
+ EnqueueOrder erased_task_enqueue_order) {
+ size_t set_index = work_queue->work_queue_set_index();
+ DCHECK_EQ(this, work_queue->work_queue_sets());
+ DCHECK_LT(set_index, enqueue_order_to_work_queue_maps_.size());
+ DCHECK(!enqueue_order_to_work_queue_maps_[set_index].empty())
+ << " set_index = " << set_index;
+ DCHECK(enqueue_order_to_work_queue_maps_[set_index].find(
+ erased_task_enqueue_order) !=
+ enqueue_order_to_work_queue_maps_[set_index].end());
+ // O(log n).
+ enqueue_order_to_work_queue_maps_[set_index].erase(erased_task_enqueue_order);
+ EnqueueOrder enqueue_order;
+ bool has_enqueue_order = work_queue->GetFrontTaskEnqueueOrder(&enqueue_order);
+ if (!has_enqueue_order)
+ return;
+ enqueue_order_to_work_queue_maps_[set_index].insert(
+ std::make_pair(enqueue_order, work_queue));
+}
+
bool WorkQueueSets::GetOldestQueueInSet(size_t set_index,
WorkQueue** out_work_queue) const {
DCHECK_LT(set_index, enqueue_order_to_work_queue_maps_.size());

Powered by Google App Engine
This is Rietveld 408576698