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 48805f790f52eecd3b9d68a541348f24dd84d72e..700a04541b6c1a6369d8d3b134b4211ece9727d9 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -4998,6 +4998,13 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   <summary>Whether a Domain Reliability upload succeeded.</summary>
 </histogram>
 
+<histogram name="DomDistiller.ArticleProcessingTime" units="milliseconds">
+  <owner>yfriedman@chromium.org</owner>
+  <summary>
+    Time spent in DomDistiller's identification of text content.
+  </summary>
+</histogram>
+
 <histogram name="DomDistiller.DistillationQuality" enum="BooleanSuccess">
   <owner>nyquist@chromium.org</owner>
   <summary>
@@ -5005,6 +5012,41 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   </summary>
 </histogram>
 
+<histogram name="DomDistiller.DistillationTotalTime" units="milliseconds">
+  <owner>yfriedman@chromium.org</owner>
+  <summary>Complete time spent in DomDistiller's processing of a page.</summary>
+</histogram>
+
+<histogram name="DomDistiller.DocumentConstructionTime" units="milliseconds">
+  <owner>yfriedman@chromium.org</owner>
+  <summary>
+    Time spent in creating DomDistiller's internal representation of the HTML
+    document.
+  </summary>
+</histogram>
+
+<histogram name="DomDistiller.FormattingTime" units="milliseconds">
+  <owner>yfriedman@chromium.org</owner>
+  <summary>
+    Time spent in DomDistiller's final processing of article content/formatting.
+  </summary>
+</histogram>
+
+<histogram name="DomDistiller.MarkupParsingTime" units="milliseconds">
+  <owner>yfriedman@chromium.org</owner>
+  <summary>
+    Time spent in DomDistiller's processing of article metadata.
+  </summary>
+</histogram>
+
+<histogram name="DomDistiller.PageDistillable" enum="BooleanEnabled">

[jwd, 2014/08/27 15:10:13]

I'd use a different enum type, maybe BooleanAvailable.

[Yaron, 2014/08/27 18:13:01]

Done.

+  <owner>yfriedman@chromium.org</owner>
+  <summary>
+    Records the number of times a page was loaded for which using DomDistiller
+    is possible.
+  </summary>
+</histogram>
+
 <histogram name="Download.AcceptRangesBytes.KBytes" units="KB">
   <owner>asanka@chromium.org</owner>
   <summary>The length of downloads for serves that accept byte ranges.</summary>