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

Unified Diff: content/renderer/scheduler/renderer_scheduler_impl.cc

Issue 922733002: scheduler: Implement task observers (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Gtestify Created 5 years, 10 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: content/renderer/scheduler/renderer_scheduler_impl.cc
diff --git a/content/renderer/scheduler/renderer_scheduler_impl.cc b/content/renderer/scheduler/renderer_scheduler_impl.cc
index 2e2da80e3bcaacc735de89fdc184792194fcfd89..1876a1ce6b953381b1843cf2b996cf8e16170e9d 100644
--- a/content/renderer/scheduler/renderer_scheduler_impl.cc
+++ b/content/renderer/scheduler/renderer_scheduler_impl.cc
@@ -48,7 +48,7 @@ RendererSchedulerImpl::RendererSchedulerImpl(
renderer_task_queue_selector_->DisableQueue(IDLE_TASK_QUEUE);
task_queue_manager_->SetAutoPump(IDLE_TASK_QUEUE, false);
// TODO(skyostil): Increase this to 4 (crbug.com/444764).
- task_queue_manager_->SetWorkBatchSize(1);
+ task_queue_manager_->SetWorkBatchSize(4);
rmcilroy 2015/02/13 16:14:00 Let's do this in a different patch (just in case w
Sami 2015/02/13 16:22:34 Oops, I wasn't actually meaning to do it here. My
for (size_t i = 0; i < TASK_QUEUE_COUNT; i++) {
task_queue_manager_->SetQueueName(
@@ -446,4 +446,16 @@ RendererSchedulerImpl::ComputeNewInputStreamState(
return INPUT_ACTIVE;
}
+void RendererSchedulerImpl::AddTaskObserver(
+ base::MessageLoop::TaskObserver* task_observer) {
+ DCHECK(main_thread_checker_.CalledOnValidThread());
+ task_queue_manager_->AddTaskObserver(task_observer);
+}
+
+void RendererSchedulerImpl::RemoveTaskObserver(
+ base::MessageLoop::TaskObserver* task_observer) {
+ DCHECK(main_thread_checker_.CalledOnValidThread());
+ task_queue_manager_->RemoveTaskObserver(task_observer);
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698