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 99f4d0c0c02f73bca653c94e5060c31dd6169973..29c81db1af0ad8fcf040b068def8ec30f5632c4d 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -45401,14 +45401,35 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
-<histogram name="Prerender.NoStatePrefetchResourceTypes"

[Alexei Svitkine (slow), 2016/09/06 19:50:23]

Instead of deleting the previous metric, please mark it as <obsolete>. This way,
old data can continue to be viewed on the dashboard.

[droger, 2016/09/06 20:43:51]

This metric is actually very recent (not even landed yet!) so nobody is looking
at it and no data has been collected.

[Alexei Svitkine (slow), 2016/09/06 21:50:52]

Okay, if it was never logged, then I'm fine with this. Thanks for clarifying.

-    enum="NoStatePrefetchResourceType">
+<histogram name="Prerender.NoStatePrefetchMainResourceRedirects">

[Alexei Svitkine (slow), 2016/09/06 21:50:52]

Please add units=""

e.g. units="redirects"

   <owner>droger@chromium.org</owner>
   <owner>mattcary@chromium.org</owner>
   <owner>pasko@chromium.org</owner>
   <summary>
-    Type of resources loaded by NoStatePrefetch, distinguishing cacheable
+    Length of the redirect chain for main resources loaded by NoStatePrefetch.
+    Recorded when the final response in the chain is received.
+  </summary>
+</histogram>
+
+<histogram name="Prerender.NoStatePrefetchResponseTypes"
+    enum="NoStatePrefetchResponseType">
+  <owner>droger@chromium.org</owner>
+  <owner>mattcary@chromium.org</owner>
+  <owner>pasko@chromium.org</owner>
+  <summary>
+    Type of responses received by NoStatePrefetch, distinguishing cacheable
     resources from no-store resources, and main resources from sub-resources.
+    Recorded when a response is received, including on each redirect.
+  </summary>
+</histogram>
+
+<histogram name="Prerender.NoStatePrefetchSubResourceRedirects">

[Alexei Svitkine (slow), 2016/09/06 21:50:52]

Ditto.

+  <owner>droger@chromium.org</owner>
+  <owner>mattcary@chromium.org</owner>
+  <owner>pasko@chromium.org</owner>
+  <summary>
+    Length of the redirect chain for sub-resources loaded by NoStatePrefetch.
+    Recorded when the final response in the chain is received.
   </summary>
 </histogram>
 
@@ -88287,11 +88308,15 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="1" label="Timed out"/>
 </enum>
 
-<enum name="NoStatePrefetchResourceType" type="int">
-  <int value="0" label="Main resource, cacheable"/>
-  <int value="1" label="Main resource, no-store"/>
-  <int value="2" label="Sub-resource, cacheable"/>
-  <int value="3" label="Sub-resource, no-store"/>
+<enum name="NoStatePrefetchResponseType" type="int">
+  <int value="0" label="Sub-resource, cacheable"/>
+  <int value="1" label="Sub-resource, no-store"/>
+  <int value="2" label="Sub-resource redirect, cacheable"/>
+  <int value="3" label="Sub-resource redirect, no-store"/>
+  <int value="4" label="Main resource, cacheable"/>
+  <int value="5" label="Main resource, no-store"/>
+  <int value="6" label="Main resource redirect, cacheable"/>
+  <int value="7" label="Main resource redirect, no-store"/>
 </enum>
 
 <enum name="NotificationActionType" type="int">
@@ -103718,7 +103743,9 @@ To add a new entry, add it with any value and run test to compute valid value.
   <affected-histogram name="Prerender.LocalPredictorTimeUntilUsed"/>
   <affected-histogram name="Prerender.NetworkBytesUsed"/>
   <affected-histogram name="Prerender.NetworkBytesWasted"/>
-  <affected-histogram name="Prerender.NoStatePrefetchResourceTypes"/>
+  <affected-histogram name="Prerender.NoStatePrefetchMainResourceRedirects"/>
+  <affected-histogram name="Prerender.NoStatePrefetchResponseTypes"/>
+  <affected-histogram name="Prerender.NoStatePrefetchSubResourceRedirects"/>
   <affected-histogram name="Prerender.PageVisitedStatus"/>
   <affected-histogram name="Prerender.PerceivedPLT"/>
   <affected-histogram name="Prerender.PerceivedPLTFirstAfterMiss"/>