Chromium Code Reviews| Index: third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc |
| diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc b/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc |
| index 9c5e33a261fef355db3ff3399cc626aa9141ce70..cd2f35e187456d9f85996d532aa4a9b27da594b2 100644 |
| --- a/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc |
| +++ b/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc |
| @@ -36,6 +36,17 @@ const double kShortIdlePeriodDurationPercentile = 50; |
| // Amount of idle time left in a frame (as a ratio of the vsync interval) above |
| // which main thread compositing can be considered fast. |
| const double kFastCompositingIdleTimeThreshold = .2; |
| + |
| +void ReportForegroundRendererTaskLoad(base::TimeTicks time, double load) { |
| + UMA_HISTOGRAM_PERCENTAGE("RendererScheduler.ForegroundRendererLoad", |
| + static_cast<int>(load * 100)); |
| +} |
| + |
| +void ReportBackgroundRendererTaskLoad(base::TimeTicks time, double load) { |
| + UMA_HISTOGRAM_PERCENTAGE("RendererScheduler.BackgroundRendererLoad", |
| + static_cast<int>(load * 100)); |
| +} |
| + |
| } // namespace |
| RendererSchedulerImpl::RendererSchedulerImpl( |
| @@ -123,6 +134,12 @@ RendererSchedulerImpl::MainThreadOnly::MainThreadOnly( |
| time_source, |
| kShortIdlePeriodDurationSampleCount, |
| kShortIdlePeriodDurationPercentile), |
| + background_renderer_load_tracker( |
| + time_source->NowTicks(), |
|
Sami
2016/08/22 14:33:16
It'd be nice to pass in the same starting time to
altimin
2016/08/22 15:36:24
Done.
|
| + base::Bind(&ReportBackgroundRendererTaskLoad)), |
| + foreground_renderer_load_tracker( |
| + time_source->NowTicks(), |
| + base::Bind(&ReportForegroundRendererTaskLoad)), |
| current_use_case(UseCase::NONE), |
| timer_queue_suspend_count(0), |
| navigation_task_expected_count(0), |
| @@ -163,6 +180,10 @@ RendererSchedulerImpl::CompositorThreadOnly::CompositorThreadOnly() |
| RendererSchedulerImpl::CompositorThreadOnly::~CompositorThreadOnly() {} |
| void RendererSchedulerImpl::Shutdown() { |
| + base::TimeTicks now = tick_clock()->NowTicks(); |
| + MainThreadOnly().background_renderer_load_tracker.RecordIdle(now); |
| + MainThreadOnly().foreground_renderer_load_tracker.RecordIdle(now); |
| + |
| throttling_helper_.reset(); |
| helper_.Shutdown(); |
| MainThreadOnly().was_shutdown = true; |
| @@ -405,6 +426,11 @@ void RendererSchedulerImpl::OnRendererBackgrounded() { |
| return; |
| MainThreadOnly().renderer_backgrounded = true; |
| + |
| + base::TimeTicks now = tick_clock()->NowTicks(); |
| + MainThreadOnly().foreground_renderer_load_tracker.Pause(now); |
| + MainThreadOnly().background_renderer_load_tracker.Resume(now); |
| + |
| if (!MainThreadOnly().timer_queue_suspension_when_backgrounded_enabled) |
| return; |
| @@ -426,6 +452,11 @@ void RendererSchedulerImpl::OnRendererForegrounded() { |
| MainThreadOnly().renderer_backgrounded = false; |
| MainThreadOnly().renderer_suspended = false; |
| + |
| + base::TimeTicks now = tick_clock()->NowTicks(); |
| + MainThreadOnly().foreground_renderer_load_tracker.Resume(now); |
| + MainThreadOnly().background_renderer_load_tracker.Pause(now); |
| + |
| suspend_timers_when_backgrounded_closure_.Cancel(); |
| ResumeTimerQueueWhenForegrounded(); |
| } |
| @@ -1413,6 +1444,10 @@ void RendererSchedulerImpl::ReportTaskTime(base::TimeTicks start_time, |
| end_time); |
| MainThreadOnly().long_task_tracker.RecordLongTask( |
| start_time, end_time - start_time); |
| + MainThreadOnly().foreground_renderer_load_tracker.RecordTaskTime(start_time, |
|
Sami
2016/08/22 14:33:16
Could we only do this for the one active one?
altimin
2016/08/22 15:36:24
Started to bail out early from ThreadLoadTracker::
|
| + end_time); |
| + MainThreadOnly().background_renderer_load_tracker.RecordTaskTime(start_time, |
| + end_time); |
|
Sami
2016/08/22 14:33:16
Maybe add a comment saying we'd really want to use
altimin
2016/08/22 15:36:24
Done.
|
| UMA_HISTOGRAM_CUSTOM_COUNTS("RendererScheduler.TaskTime", |
| (end_time - start_time).InMicroseconds(), 1, |
| 1000000, 50); |