EQT: Record the max queueing time from navigation start to navigation away

third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc

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 1578d3e96fd35054fb08c96a5aeb94ad7f759ff7..bc61230a29919376e43523d4543eaa1fa6a5666e 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
@@ -220,6 +220,7 @@ RendererSchedulerImpl::MainThreadOnly::MainThreadOnly(
       use_virtual_time(false),
       is_audio_playing(false),
       virtual_time_paused(false),
+      has_navigated(false),
       rail_mode_observer(nullptr),
       wake_up_budget_pool(nullptr),
       task_duration_reporter("RendererScheduler.TaskDurationPerQueueType2"),
@@ -1646,6 +1647,25 @@ void RendererSchedulerImpl::RemovePendingNavigation(NavigatingFrameType type) {
   }
 }
 
+void RendererSchedulerImpl::OnCommitProvisionalLoad() {

[Charlie Reis, 2017/06/02 22:38:36]

nit: This should be after OnNavigationStarted (to match order in .h file).

[Liquan (Max) Gu, 2017/06/02 23:32:06]

Done.

+  // Initialize |max_queueing_time_metric| lazily so that
+  // |SingleSampleMetricsFactory::SetFactory()| is called before
+  // |SingleSampleMetricsFactory::Get()|
+  if (!GetMainThreadOnly().max_queueing_time_metric) {
+    GetMainThreadOnly().max_queueing_time_metric =
+        CreateMaxQueueingTimeMetric();
+  }
+  GetMainThreadOnly().max_queueing_time_metric.reset();
+  GetMainThreadOnly().max_queueing_time = base::TimeDelta();
+  GetMainThreadOnly().has_navigated = true;
+}
+
+std::unique_ptr<base::SingleSampleMetric>
+RendererSchedulerImpl::CreateMaxQueueingTimeMetric() {
+  return base::SingleSampleMetricsFactory::Get()->CreateCustomCountsMetric(
+      "RendererScheduler.MaxQueueingTime", 1, 10000, 50);
+}
+
 void RendererSchedulerImpl::OnNavigationStarted() {
   TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"),
                "RendererSchedulerImpl::OnNavigationStarted");
@@ -1914,6 +1934,18 @@ void RendererSchedulerImpl::RemoveTaskTimeObserver(
 void RendererSchedulerImpl::OnQueueingTimeForWindowEstimated(
     base::TimeDelta queueing_time,
     base::TimeTicks window_start_time) {
+  if (GetMainThreadOnly().has_navigated) {
+    if (GetMainThreadOnly().max_queueing_time < queueing_time) {
+      if (!GetMainThreadOnly().max_queueing_time_metric) {
+        GetMainThreadOnly().max_queueing_time_metric =
+            CreateMaxQueueingTimeMetric();
+      }
+      GetMainThreadOnly().max_queueing_time_metric->SetSample(
+          queueing_time.InMilliseconds());
+      GetMainThreadOnly().max_queueing_time = queueing_time;
+    }
+  }
+
   // RendererScheduler reports the queueing time once per window's duration.
   //          |stepEQT|stepEQT|stepEQT|stepEQT|stepEQT|stepEQT|
   // Report:  |-------window EQT------|