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

Side by Side Diff: third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc

Issue 2319053004: [Reland] Make canceling Timers fast. (Closed)
Patch Set: Rebased 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "platform/scheduler/base/task_queue_manager.h" 5 #include "platform/scheduler/base/task_queue_manager.h"
6 6
7 #include <queue> 7 #include <queue>
8 #include <set> 8 #include <set>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 257 matching lines...) Expand 10 before | Expand all | Expand 10 after
268 268
269 void TaskQueueManager::DidQueueTask( 269 void TaskQueueManager::DidQueueTask(
270 const internal::TaskQueueImpl::Task& pending_task) { 270 const internal::TaskQueueImpl::Task& pending_task) {
271 task_annotator_.DidQueueTask("TaskQueueManager::PostTask", pending_task); 271 task_annotator_.DidQueueTask("TaskQueueManager::PostTask", pending_task);
272 } 272 }
273 273
274 TaskQueueManager::ProcessTaskResult TaskQueueManager::ProcessTaskFromWorkQueue( 274 TaskQueueManager::ProcessTaskResult TaskQueueManager::ProcessTaskFromWorkQueue(
275 internal::WorkQueue* work_queue) { 275 internal::WorkQueue* work_queue) {
276 DCHECK(main_thread_checker_.CalledOnValidThread()); 276 DCHECK(main_thread_checker_.CalledOnValidThread());
277 scoped_refptr<DeletionSentinel> protect(deletion_sentinel_); 277 scoped_refptr<DeletionSentinel> protect(deletion_sentinel_);
278 internal::TaskQueueImpl::Task pending_task =
279 work_queue->TakeTaskFromWorkQueue();
280
281 // It's possible the task was canceled, if so bail out.
282 if (pending_task.task.IsCancelled())
283 return ProcessTaskResult::EXECUTED;
284
278 internal::TaskQueueImpl* queue = work_queue->task_queue(); 285 internal::TaskQueueImpl* queue = work_queue->task_queue();
279
280 if (queue->GetQuiescenceMonitored()) 286 if (queue->GetQuiescenceMonitored())
281 task_was_run_on_quiescence_monitored_queue_ = true; 287 task_was_run_on_quiescence_monitored_queue_ = true;
282 288
283 internal::TaskQueueImpl::Task pending_task =
284 work_queue->TakeTaskFromWorkQueue();
285 if (!pending_task.nestable && delegate_->IsNested()) { 289 if (!pending_task.nestable && delegate_->IsNested()) {
286 // Defer non-nestable work to the main task runner. NOTE these tasks can be 290 // Defer non-nestable work to the main task runner. NOTE these tasks can be
287 // arbitrarily delayed so the additional delay should not be a problem. 291 // arbitrarily delayed so the additional delay should not be a problem.
288 // TODO(skyostil): Figure out a way to not forget which task queue the 292 // TODO(skyostil): Figure out a way to not forget which task queue the
289 // task is associated with. See http://crbug.com/522843. 293 // task is associated with. See http://crbug.com/522843.
290 delegate_->PostNonNestableTask(pending_task.posted_from, pending_task.task); 294 delegate_->PostNonNestableTask(pending_task.posted_from,
295 std::move(pending_task.task));
291 return ProcessTaskResult::DEFERRED; 296 return ProcessTaskResult::DEFERRED;
292 } 297 }
293 298
294 MaybeRecordTaskDelayHistograms(pending_task, queue); 299 MaybeRecordTaskDelayHistograms(pending_task, queue);
295 300
296 TRACE_TASK_EXECUTION("TaskQueueManager::ProcessTaskFromWorkQueue", 301 TRACE_TASK_EXECUTION("TaskQueueManager::ProcessTaskFromWorkQueue",
297 pending_task); 302 pending_task);
298 if (queue->GetShouldNotifyObservers()) { 303 if (queue->GetShouldNotifyObservers()) {
299 FOR_EACH_OBSERVER(base::MessageLoop::TaskObserver, task_observers_, 304 FOR_EACH_OBSERVER(base::MessageLoop::TaskObserver, task_observers_,
300 WillProcessTask(pending_task)); 305 WillProcessTask(pending_task));
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
432 DCHECK(main_thread_checker_.CalledOnValidThread()); 437 DCHECK(main_thread_checker_.CalledOnValidThread());
433 DCHECK(!work_queue->Empty()); 438 DCHECK(!work_queue->Empty());
434 if (observer_) { 439 if (observer_) {
435 observer_->OnTriedToExecuteBlockedTask(*work_queue->task_queue(), 440 observer_->OnTriedToExecuteBlockedTask(*work_queue->task_queue(),
436 *work_queue->GetFrontTask()); 441 *work_queue->GetFrontTask());
437 } 442 }
438 } 443 }
439 444
440 } // namespace scheduler 445 } // namespace scheduler
441 } // namespace blink 446 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698