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

Side by Side Diff: components/scheduler/child/scheduler_helper.cc

Issue 2113893006: POC: Instrument task time tracker Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: undo delegate delete 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "components/scheduler/child/scheduler_helper.h" 5 #include "components/scheduler/child/scheduler_helper.h"
6 6
7 #include "base/time/default_tick_clock.h" 7 #include "base/time/default_tick_clock.h"
8 #include "base/trace_event/trace_event.h" 8 #include "base/trace_event/trace_event.h"
9 #include "base/trace_event/trace_event_argument.h" 9 #include "base/trace_event/trace_event_argument.h"
10 #include "components/scheduler/base/task_queue_impl.h" 10 #include "components/scheduler/base/task_queue_impl.h"
11 #include "components/scheduler/child/scheduler_tqm_delegate.h" 11 #include "components/scheduler/child/scheduler_tqm_delegate.h"
12 12
13 namespace scheduler { 13 namespace scheduler {
14 14
15 SchedulerHelper::SchedulerHelper( 15 SchedulerHelper::SchedulerHelper(
16 scoped_refptr<SchedulerTqmDelegate> task_queue_manager_delegate, 16 scoped_refptr<SchedulerTqmDelegate> task_queue_manager_delegate,
17 const char* tracing_category, 17 const char* tracing_category,
18 const char* disabled_by_default_tracing_category, 18 const char* disabled_by_default_tracing_category,
19 const char* disabled_by_default_verbose_tracing_category) 19 const char* disabled_by_default_verbose_tracing_category)
20 : task_queue_manager_delegate_(task_queue_manager_delegate), 20 : task_queue_manager_delegate_(task_queue_manager_delegate),
21 task_time_tracker_(new TaskTimeTracker()),
21 task_queue_manager_( 22 task_queue_manager_(
22 new TaskQueueManager(task_queue_manager_delegate, 23 new TaskQueueManager(task_queue_manager_delegate,
23 tracing_category, 24 tracing_category,
24 disabled_by_default_tracing_category, 25 disabled_by_default_tracing_category,
25 disabled_by_default_verbose_tracing_category)), 26 disabled_by_default_verbose_tracing_category,
27 task_time_tracker_.get())),
26 control_task_runner_(NewTaskQueue( 28 control_task_runner_(NewTaskQueue(
27 TaskQueue::Spec("control_tq") 29 TaskQueue::Spec("control_tq")
28 .SetWakeupPolicy(TaskQueue::WakeupPolicy::DONT_WAKE_OTHER_QUEUES) 30 .SetWakeupPolicy(TaskQueue::WakeupPolicy::DONT_WAKE_OTHER_QUEUES)
29 .SetShouldNotifyObservers(false))), 31 .SetShouldNotifyObservers(false))),
30 control_after_wakeup_task_runner_(NewTaskQueue( 32 control_after_wakeup_task_runner_(NewTaskQueue(
31 TaskQueue::Spec("control_after_wakeup_tq") 33 TaskQueue::Spec("control_after_wakeup_tq")
32 .SetPumpPolicy(TaskQueue::PumpPolicy::AFTER_WAKEUP) 34 .SetPumpPolicy(TaskQueue::PumpPolicy::AFTER_WAKEUP)
33 .SetWakeupPolicy(TaskQueue::WakeupPolicy::DONT_WAKE_OTHER_QUEUES) 35 .SetWakeupPolicy(TaskQueue::WakeupPolicy::DONT_WAKE_OTHER_QUEUES)
34 .SetShouldNotifyObservers(false))), 36 .SetShouldNotifyObservers(false))),
35 default_task_runner_(NewTaskQueue(TaskQueue::Spec("default_tq") 37 default_task_runner_(NewTaskQueue(TaskQueue::Spec("default_tq")
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 base::MessageLoop::TaskObserver* task_observer) { 116 base::MessageLoop::TaskObserver* task_observer) {
115 CheckOnValidThread(); 117 CheckOnValidThread();
116 if (task_queue_manager_) 118 if (task_queue_manager_)
117 task_queue_manager_->RemoveTaskObserver(task_observer); 119 task_queue_manager_->RemoveTaskObserver(task_observer);
118 } 120 }
119 121
120 void SchedulerHelper::SetObserver(Observer* observer) { 122 void SchedulerHelper::SetObserver(Observer* observer) {
121 CheckOnValidThread(); 123 CheckOnValidThread();
122 observer_ = observer; 124 observer_ = observer;
123 DCHECK(task_queue_manager_); 125 DCHECK(task_queue_manager_);
126 DCHECK(task_time_tracker_);
124 task_queue_manager_->SetObserver(this); 127 task_queue_manager_->SetObserver(this);
128 task_time_tracker_->SetObserver(this);
125 } 129 }
126 130
127 RealTimeDomain* SchedulerHelper::real_time_domain() const { 131 RealTimeDomain* SchedulerHelper::real_time_domain() const {
128 CheckOnValidThread(); 132 CheckOnValidThread();
129 DCHECK(task_queue_manager_); 133 DCHECK(task_queue_manager_);
130 return task_queue_manager_->real_time_domain(); 134 return task_queue_manager_->real_time_domain();
131 } 135 }
132 136
133 void SchedulerHelper::RegisterTimeDomain(TimeDomain* time_domain) { 137 void SchedulerHelper::RegisterTimeDomain(TimeDomain* time_domain) {
134 CheckOnValidThread(); 138 CheckOnValidThread();
(...skipping 19 matching lines...) Expand all
154 if (observer_) 158 if (observer_)
155 observer_->OnTriedToExecuteBlockedTask(queue, task); 159 observer_->OnTriedToExecuteBlockedTask(queue, task);
156 } 160 }
157 161
158 TaskQueue* SchedulerHelper::CurrentlyExecutingTaskQueue() const { 162 TaskQueue* SchedulerHelper::CurrentlyExecutingTaskQueue() const {
159 if (!task_queue_manager_) 163 if (!task_queue_manager_)
160 return nullptr; 164 return nullptr;
161 return task_queue_manager_->currently_executing_task_queue(); 165 return task_queue_manager_->currently_executing_task_queue();
162 } 166 }
163 167
168 // TaskTimeTracker::TaskTimeObserver implementation:
169 void SchedulerHelper::OnLongTask(base::TimeDelta long_task_time) {
170 // Record long task
171 // will be queried from RendererSchedulerImpl in BeginMainFrame
172 }
173
164 } // namespace scheduler 174 } // namespace scheduler
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698