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

Unified Diff: components/scheduler/child/scheduler_helper.cc

Issue 1898233002: Report expected task queueing time via UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix perf test Created 4 years, 5 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: components/scheduler/child/scheduler_helper.cc
diff --git a/components/scheduler/child/scheduler_helper.cc b/components/scheduler/child/scheduler_helper.cc
index 43944359e0497c0b997600f46f027efed3516ea1..64dbecc34f20e4f6ab83029ecc66a682dd77d04b 100644
--- a/components/scheduler/child/scheduler_helper.cc
+++ b/components/scheduler/child/scheduler_helper.cc
@@ -22,7 +22,8 @@ SchedulerHelper::SchedulerHelper(
new TaskQueueManager(task_queue_manager_delegate,
tracing_category,
disabled_by_default_tracing_category,
- disabled_by_default_verbose_tracing_category)),
+ disabled_by_default_verbose_tracing_category,
+ &task_time_tracker_)),
control_task_runner_(NewTaskQueue(
TaskQueue::Spec("control_tq")
.SetWakeupPolicy(TaskQueue::WakeupPolicy::DONT_WAKE_OTHER_QUEUES)
@@ -32,8 +33,8 @@ SchedulerHelper::SchedulerHelper(
.SetPumpPolicy(TaskQueue::PumpPolicy::AFTER_WAKEUP)
.SetWakeupPolicy(TaskQueue::WakeupPolicy::DONT_WAKE_OTHER_QUEUES)
.SetShouldNotifyObservers(false))),
- default_task_runner_(NewTaskQueue(TaskQueue::Spec("default_tq")
- .SetShouldMonitorQuiescence(true))),
+ default_task_runner_(NewTaskQueue(
+ TaskQueue::Spec("default_tq").SetShouldMonitorQuiescence(true))),
observer_(nullptr),
tracing_category_(tracing_category),
disabled_by_default_tracing_category_(
@@ -122,6 +123,7 @@ void SchedulerHelper::SetObserver(Observer* observer) {
observer_ = observer;
DCHECK(task_queue_manager_);
task_queue_manager_->SetObserver(this);
+ task_time_tracker_.SetObserver(this);
}
RealTimeDomain* SchedulerHelper::real_time_domain() const {
@@ -161,4 +163,8 @@ TaskQueue* SchedulerHelper::CurrentlyExecutingTaskQueue() const {
return task_queue_manager_->currently_executing_task_queue();
}
+// TaskTimeTracker::TaskTimeObserver implementation:
+void SchedulerHelper::OnQueueingTimeForWindowEstimated(
+ base::TimeDelta queueing_time) {}
+
} // namespace scheduler

Powered by Google App Engine
This is Rietveld 408576698