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 c5a6a1051d3ab145f47801f6850103b3fd4ca7ec..6da4238dddc2e95f7477926fa1769e6cf489adca 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -13931,6 +13931,11 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Download.HttpResponseCode">

[Ilya Sherman, 2017/05/04 06:06:50]

Is there an enum that's appropriate to associate with this histogram?

[xingliu, 2017/05/04 16:52:41]

No, this metric logs all the http response code. "Net.HttpResponseCode" also do
this, but it collects the response code for all the network quests, where we
need the code for download requests.
 

[Ilya Sherman, 2017/05/04 19:39:24]

It looks like the HttpResponseCode enum is appropriate for recording HTTP
response codes.  Am I misunderstanding?

Perhaps to clarify: I'm simply suggesting adding the attribute
enum="HttpResponseCode" to this histogram.  I'm not suggesting making any
changes to the C++ code.

+  <owner>xingliu@chromium.org</owner>
+  <summary>The count of HTTP Response codes for download requests.</summary>
+</histogram>
+
 <histogram name="Download.InterruptedAtEndError" enum="NetErrorCodes">
   <owner>asanka@chromium.org</owner>
   <summary>