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 7400ff4b9423df282aba7d8ddb4f26d2b7c6b71b..9c1dc5d3ab91cf28d5b30da99f2943eb446168c8 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -649,6 +649,15 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <summary>The percentage of total storage downloads consume.</summary>
 </histogram>
 
+<histogram name="Android.DownloadManager.StorageInfo" enum="BooleanEnabled">

[Ilya Sherman, 2017/05/12 00:08:27]

nit: I'd name this histogram "Android.DownloadManager.ShowStorageInfo"

[shaktisahu, 2017/05/12 00:34:08]

Done.

+  <owner>shaktisahu@chromium.org</owner>
+  <summary>
+    Recorded when the user clicks the info button on download home to toggle the
+    storage info. Enabled is recorded when storage info is shown and disabled
+    when it is hidden.

[Ilya Sherman, 2017/05/12 00:08:27]

I think this second sentence is describing how to interpret the enum labels?  If
so, why not just use an enum with clearer labels?

[shaktisahu, 2017/05/12 00:34:08]

Done. Using BooleanVisible.

+  </summary>
+</histogram>
+
 <histogram name="Android.HistoryPage.OpenSelected">
   <owner>twellington@chromium.org</owner>
   <summary>