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 e0c783d57f2fa3e43afc16272c3b001a3ab36be7..32ce50ec7dedfb00c2a5c2ac03bb8b1203bf6735 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -45247,6 +45247,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Prerender.NoStatePrefetchMainResourceRedirects">
+  <owner>droger@chromium.org</owner>
+  <owner>mattcary@chromium.org</owner>
+  <owner>pasko@chromium.org</owner>
+  <summary>
+    Length of the redirect chain for main resources loaded by NoStatePrefetch.
+  </summary>
+</histogram>
+
 <histogram name="Prerender.NoStatePrefetchResourceTypes"
     enum="NoStatePrefetchResourceType">
   <owner>droger@chromium.org</owner>
@@ -45258,6 +45267,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Prerender.NoStatePrefetchSubResourceRedirects">
+  <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.
+  </summary>
+</histogram>
+
 <histogram name="Prerender.OmniboxNavigationsCouldPrerender">
   <owner>pasko@chromium.org</owner>
   <summary>
@@ -87819,10 +87837,14 @@ To add a new entry, add it with any value and run test to compute valid value.
 </enum>
 
 <enum name="NoStatePrefetchResourceType" type="int">

[pasko, 2016/09/02 16:34:48]

this is used for a histogram called "Prerender.NoStatePrefetchResourceTypes"? I
think I mistakenly assumed in another change that this histogram is recorded
once per resource (i.e. once per redirect chain), but in a closer look I am
seeing these counts would be recorded once at having the 1st byte of the last
request in the chain and once at every redirect. Is that right?

In this case the I'd like to take my words back .. this name should reflect
counts and probably should be rephrased in terms of URL requests and not
resources, right?

[droger, 2016/09/05 12:38:47]

You're right. Before this CL it was called once per resource, but now it is
called once per response (including redirects).
Sure, renamed by changing *ResourceType into *ResponseType everywhere.

-  <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"/>
+  <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">
@@ -103217,7 +103239,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.NoStatePrefetchMainResourceRedirects"/>
   <affected-histogram name="Prerender.NoStatePrefetchResourceTypes"/>
+  <affected-histogram name="Prerender.NoStatePrefetchSubResourceRedirects"/>
   <affected-histogram name="Prerender.PageVisitedStatus"/>
   <affected-histogram name="Prerender.PerceivedPLT"/>
   <affected-histogram name="Prerender.PerceivedPLTFirstAfterMiss"/>