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

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

Issue 2886943003: [Offline Pages] Adding missing image/CSS detection in FrameSerializer. (Closed)
Patch Set: reducing the number of buckets of percentage to 21. Created 3 years, 6 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
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 9bbb211bc1b4c37f4efbc91ba3ac6845eaaef1c7..42b84dc670e767a5accdcfff6f73ddcb7e7b5b64 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -50047,6 +50047,35 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="PageSerialization.ProblemDetection.LoadedCSSPercentage"
+ units="%">
+ <owner>romax@chromium.org</owner>
+ <summary>
+ Percentage of loaded CSS elements in the main frame at the time of
+ serialization.
+ </summary>
+</histogram>
+
+<histogram name="PageSerialization.ProblemDetection.MissingImagePercentage"
+ units="%">
+ <owner>romax@chromium.org</owner>
+ <summary>
+ Percentage of loaded images in the main frame at the time of serialization.
Łukasz Anforowicz 2017/06/07 21:11:35 It seems that "loaded" (above) vs "missing" (in th
romax 2017/06/08 02:25:22 Done.
+ </summary>
+</histogram>
+
+<histogram name="PageSerialization.ProblemDetection.TotalCSSCount">
+ <owner>romax@chromium.org</owner>
+ <summary>
+ Total number of CSS elements in the main frame for serialization.
+ </summary>
+</histogram>
+
+<histogram name="PageSerialization.ProblemDetection.TotalImageCount">
+ <owner>romax@chromium.org</owner>
+ <summary>Total number of images in the main frame for serialization.</summary>
+</histogram>
+
<histogram name="PageSerialization.SerializationTime.CSSElement"
units="microseconds">
<owner>carlosk@chromium.org</owner>

Powered by Google App Engine
This is Rietveld 408576698