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 71c65a7709a050c05e316c45e6510a9e3726ee60..28cef7ad4d02582e0ea09177b0223a2efa10d19d 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -13217,6 +13217,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 </histogram>
 
 <histogram name="Download.ContentType" enum="DownloadContentType">
+  <obsolete>
+    Deprecated 03/2017.

[asanka, 2017/03/23 23:54:10]

Add a note here pointing at Download.Start.ContentType as the new hotness.

+  </obsolete>
   <owner>asanka@chromium.org</owner>
   <summary>Content types that are downloaded.</summary>
 </histogram>
@@ -13762,6 +13765,19 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Download.Start.ContentType" enum="DownloadContentType">
+  <owner>shaktisahu@chromium.org</owner>
+  <summary>Content types of the downloads that are started.</summary>

[asanka, 2017/03/23 23:54:10]

I'd leave out the "that are started" part. In isolation it's confusing.

At some point it's probably worth going through and making it explicit that some
metrics are per request while some others are per download. The concepts
diverged when resumption rolled out and we didn't clean up the descriptions to
keep up. My bad. :-(

+</histogram>
+
+<histogram name="Download.Start.ContentType.NormalProfile"
+    enum="DownloadContentType">
+  <owner>shaktisahu@chromium.org</owner>
+  <summary>
+    Content types of the downloads that are started in non-incognito profile.
+  </summary>
+</histogram>
+
 <histogram name="Download.TargetConnectionSecurity"
     enum="DownloadConnectionSecurity">
   <owner>jialiul@chromium.org</owner>
@@ -87880,6 +87896,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <int value="16" label="No WebContents at interruption"/>
   <int value="17" label="Supports ranges and strong validation"/>
   <int value="18" label="Uses parallel requests"/>
+  <int value="19" label="New downloads"/>
+  <int value="20" label="New downloads (normal profile only)"/>
+  <int value="21" label="Completed (normal profile only)"/>
 </enum>
 
 <enum name="DownloadDatabaseRecordDroppedType" type="int">