Generalize abort tracking to page end state tracking

tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index f7fd8937c039f19f0327adf824671dab93cf2849..a5d0e576969984280cb0a7bf49c7f37fe655679d 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -44074,6 +44074,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 
 <histogram base="true" name="PageLoad.Experimental.AbortTiming.ClientRedirect"
     units="ms">
+  <obsolete>
+    Deprecated in favor of PageLoad.Internal.ClientRedirect.*.
+  </obsolete>
   <owner>csharrison@chromium.org</owner>
   <summary>
     This metric is still experimental and not yet ready to be relied upon.
@@ -44215,6 +44218,42 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="PageLoad.Experimental.PageTiming.FirstPaintToFirstBackground"
+    units="ms">
+  <owner>bmcquade@chromium.org</owner>
+  <summary>
+    Measures the total time the page load was active after first paint, up until
+    being backgrounded, for page loads that started in the foreground.

[Ilya Sherman, 2017/02/22 01:33:13]

nit: I'd add "and were backgrounded prior to page load completing" (or something
like that) to the end of this sentence.  And, to keep the sentence from getting
too complex, I might split the sentence into two, like so: "... up until being
backgrounded.  Recorded only for page loads that ..."

+  </summary>
+</histogram>
+
+<histogram name="PageLoad.Experimental.PageTiming.FirstPaintToPageEnd"
+    units="ms">
+  <owner>bmcquade@chromium.org</owner>
+  <summary>
+    Measures the total time the page load was active after first paint, for page
+    loads that spend the entire time in the foreground.
+  </summary>
+</histogram>
+
+<histogram name="PageLoad.Experimental.PageTiming.NavigationToFirstBackground"
+    units="ms">
+  <owner>bmcquade@chromium.org</owner>
+  <summary>
+    Measures the total time the page load was active, up until being
+    backgrounded, for page loads that started in the foreground.
+  </summary>
+</histogram>
+
+<histogram name="PageLoad.Experimental.PageTiming.NavigationToPageEnd"
+    units="ms">
+  <owner>bmcquade@chromium.org</owner>
+  <summary>
+    Measures the total time the page load was active, for page loads that spend
+    the entire time in the foreground.
+  </summary>
+</histogram>
+
 <histogram
     name="PageLoad.Experimental.PaintTiming.FirstMeaningfulPaintSignalStatus"
     enum="FirstMeaningfulPaintSignalStatus">
@@ -44452,6 +44491,27 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="PageLoad.PageTiming.NavigationToFailedProvisionalLoad"
+    units="ms">
+  <owner>bmcquade@chromium.org</owner>
+  <owner>shivanisha@chromium.org</owner>
+  <summary>
+    Measures the time from navigation timing's navigation start to the time the
+    provisional load failed. Only measures provisional loads that failed in the
+    foreground.
+  </summary>
+</histogram>
+
+<histogram name="PageLoad.PageTiming.NavigationToFirstForeground" units="ms">
+  <owner>bmcquade@chromium.org</owner>
+  <owner>csharrison@chromium.org</owner>
+  <summary>
+    Measures the time from navigation timing's navigation start to the time the
+    user first foregrounds an initially backgrounded tab. Only measures
+    navigations that started in the background.
+  </summary>
+</histogram>
+
 <histogram name="PageLoad.PaintTiming.ForegroundToFirstPaint" units="ms">
   <owner>pkotwicz@chromium.org</owner>
   <summary>
@@ -44667,6 +44727,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 </histogram>
 
 <histogram name="PageLoad.Timing2.NavigationToFailedProvisionalLoad" units="ms">
+  <obsolete>
+    deprecated in favor of PageLoad.PageTiming.NavigationToFailedProvisionalLoad
+  </obsolete>
   <owner>bmcquade@chromium.org</owner>
   <owner>shivanisha@chromium.org</owner>
   <summary>
@@ -44702,6 +44765,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 </histogram>
 
 <histogram name="PageLoad.Timing2.NavigationToFirstForeground" units="ms">
+  <obsolete>
+    deprecated in favor of PageLoad.PageTiming.NavigationToFirstForeground
+  </obsolete>
   <owner>bmcquade@chromium.org</owner>
   <owner>csharrison@chromium.org</owner>
   <summary>
@@ -95397,7 +95463,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <int value="2" label="IPC received from a frame we navigated away from"/>
   <int value="3" label="IPC received from a bad URL scheme"/>
   <int value="4" label="No IPCs received for this navigation"/>
-  <int value="5" label="Abort reported before navigation start"/>
+  <int value="5" label="Page end reported before navigation start"/>
   <int value="6"
       label="Multiple aborted provisional loads at navigation start"/>
   <int value="7"
@@ -95405,6 +95471,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
              (deprecated)"/>
   <int value="8" label="Inter process TimeTicks skew"/>
   <int value="9" label="No commit or failed provisional load received"/>
+  <int value="10" label="No page load end time recorded"/>
 </enum>
 
 <enum name="InterruptReason" type="int">
@@ -117721,6 +117788,16 @@ value.
   <affected-histogram name="PageLoad.ParseTiming.NavigationToParseStart"/>
 </histogram_suffixes>
 
+<histogram_suffixes name="PageLoadMetricsNoEndTime" separator=".">
+  <suffix name="NoEndTime"
+      label="The page load had no recorded end time, so an end time was
+             synthesized at the time the page end notification was processed."/>
+  <affected-histogram
+      name="PageLoad.Experimental.PageTiming.FirstPaintToPageEnd"/>
+  <affected-histogram
+      name="PageLoad.Experimental.PageTiming.NavigationToPageEnd"/>
+</histogram_suffixes>
+
 <histogram_suffixes name="PageLoadMetricsUserGesture" separator=".">
   <suffix name="UserGesture"
       label="Restricted to pages loaded via a user gesture."/>