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 6b101a8e6c04a627e87e153cc281396824dc7fc2..2f6e7c2a2628e8f9ff9e8e8628c56b041a231147 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -26018,6 +26018,14 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   </summary>
 </histogram>
 
+<histogram name="SBIRS.DownloadCollectionTime" units="milliseconds">
+  <owner>grt@google.com</owner>
+  <summary>
+    The elapsed time to collect the most recent binary download for a safe

[Mark P, 2014/06/30 23:11:06]

This sounds awkward.  What does "collect" mean in this context?

Also, what does most recent mean in this context?  Do you not report this on
some downloads?

FWIW, the histogram name implies the description should simply be

The elapsed time to download a safe browsing incident report.

+    browsing incident report.
+  </summary>
+</histogram>
+
 <histogram name="SBIRS.DroppedIncident" enum="IncidentType">
   <owner>grt@google.com</owner>
   <summary>
@@ -47206,6 +47214,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   <int value="3" label="Cancelled"/>
   <int value="4" label="RequestFailed"/>
   <int value="5" label="InvalidResponse"/>
+  <int value="6" label="NoDownload">No download found to report</int>

[Mark P, 2014/06/30 23:11:06]

This doesn't seem consistent with the description, which implies that an upload
took place.

Please update the description if you think this is still the right way to report
this information.

 </enum>
 
 <enum name="UrlResolutionResult" type="int">