Commit f2b39a21 authored by Alexander Timin's avatar Alexander Timin Committed by Commit Bot

[scheduler] Add keep-active in background metrics.

Measure the amount of work done in background when keep-active signal is
present. This is to allow us to test the theory that we stopped stopping
timers in background after 5 minutes on Android due to widespread
keep-active signal.

R=panicker@chromium.org,rkaplow@chromium.org
BUG=823482

Change-Id: I653016d795a1903ea9d943e3c2b962ab9c1ba088
Reviewed-on: https://chromium-review.googlesource.com/973225Reviewed-by: default avatarShubhie Panicker <panicker@chromium.org>
Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
Commit-Queue: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545126}
parent 1f0394f5
...@@ -83,37 +83,30 @@ class PLATFORM_EXPORT RendererMetricsHelper : public MetricsHelper { ...@@ -83,37 +83,30 @@ class PLATFORM_EXPORT RendererMetricsHelper : public MetricsHelper {
using TaskDurationPerQueueTypeMetricReporter = using TaskDurationPerQueueTypeMetricReporter =
TaskDurationMetricReporter<MainThreadTaskQueue::QueueType>; TaskDurationMetricReporter<MainThreadTaskQueue::QueueType>;
TaskDurationPerQueueTypeMetricReporter per_queue_type_task_duration_reporter_; struct PerQueueTypeDurationReporters {
TaskDurationPerQueueTypeMetricReporter PerQueueTypeDurationReporters();
foreground_per_queue_type_task_duration_reporter_;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter overall;
foreground_first_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter foreground;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter foreground_first_minute;
foreground_second_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter foreground_second_minute;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter foreground_third_minute;
foreground_third_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter foreground_after_third_minute;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter background;
foreground_after_third_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter background_first_minute;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter background_second_minute;
background_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter background_third_minute;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter background_fourth_minute;
background_first_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter background_fifth_minute;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter background_after_fifth_minute;
background_second_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter
TaskDurationPerQueueTypeMetricReporter background_keep_active_after_fifth_minute;
background_third_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter hidden;
TaskDurationPerQueueTypeMetricReporter TaskDurationPerQueueTypeMetricReporter visible;
background_fourth_minute_per_queue_type_task_duration_reporter_; TaskDurationPerQueueTypeMetricReporter hidden_music;
TaskDurationPerQueueTypeMetricReporter };
background_fifth_minute_per_queue_type_task_duration_reporter_;
TaskDurationPerQueueTypeMetricReporter PerQueueTypeDurationReporters per_queue_type_reporters_;
background_after_fifth_minute_per_queue_type_task_duration_reporter_;
TaskDurationPerQueueTypeMetricReporter
hidden_per_queue_type_task_duration_reporter_;
TaskDurationPerQueueTypeMetricReporter
visible_per_queue_type_task_duration_reporter_;
TaskDurationPerQueueTypeMetricReporter
hidden_music_per_queue_type_task_duration_reporter_;
TaskDurationMetricReporter<FrameStatus> per_frame_status_duration_reporter_; TaskDurationMetricReporter<FrameStatus> per_frame_status_duration_reporter_;
......
...@@ -113588,6 +113588,10 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries. ...@@ -113588,6 +113588,10 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
label="Time spent in tasks of a particular type starting from the sixth label="Time spent in tasks of a particular type starting from the sixth
minute after backgrounding the renderer. The renderer is minute after backgrounding the renderer. The renderer is
expected to be mostly idle during this period."/> expected to be mostly idle during this period."/>
<suffix name="Background.KeepAlive.AfterFifthMinute"
label="Time spent in tasks of a particular type starting from the sixth
minute after backgrounding the renderer when keep-alive signal
is present."/>
<suffix name="Foreground" <suffix name="Foreground"
label="Time spent in tasks of a particular type when the renderer is in label="Time spent in tasks of a particular type when the renderer is in
the foreground. Please note that individual tabs in this the foreground. Please note that individual tabs in this
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment