Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(570)

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 501553002: Record histograms for timing information in DomDistiller. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« no previous file with comments | « components/dom_distiller/core/distiller_page.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 9557f88eda48c9b0501c73e3e7069a7b0f223e62..c9a48ac3fa1707c6bc98665ec1e39c08853c0d08 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -5005,6 +5005,48 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="DomDistiller.PageDistillable" enum="BooleanDistillable">
+ <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="DomDistiller.Time.ArticleProcessing" units="milliseconds">
+ <owner>yfriedman@chromium.org</owner>
+ <summary>
+ Time spent in DomDistiller's identification of text content.
+ </summary>
+</histogram>
+
+<histogram name="DomDistiller.Time.DistillationTotal" units="milliseconds">
+ <owner>yfriedman@chromium.org</owner>
+ <summary>Complete time spent in DomDistiller's processing of a page.</summary>
+</histogram>
+
+<histogram name="DomDistiller.Time.DocumentConstruction" 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.Time.Formatting" units="milliseconds">
+ <owner>yfriedman@chromium.org</owner>
+ <summary>
+ Time spent in DomDistiller's final processing of article content/formatting.
+ </summary>
+</histogram>
+
+<histogram name="DomDistiller.Time.MarkupParsing" units="milliseconds">
+ <owner>yfriedman@chromium.org</owner>
+ <summary>
+ Time spent in DomDistiller's processing of article metadata.
+ </summary>
+</histogram>
+
<histogram name="DOMStorage.clear" units="milliseconds">
<owner>michaeln@chromium.org</owner>
<summary>
@@ -37900,6 +37942,11 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<int value="1" label="Did evict"/>
</enum>
+<enum name="BooleanDistillable" type="int">
+ <int value="0" label="Not distillable"/>
+ <int value="1" label="Distillable"/>
+</enum>
+
<enum name="BooleanDuplicate" type="int">
<int value="0" label="Not Duplicate"/>
<int value="1" label="Duplicate"/>
« no previous file with comments | « components/dom_distiller/core/distiller_page.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698