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

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

Issue 2581043003: Add page-level aggregation of SubresourceFilter time metrics. (Closed)
Patch Set: Address comments of engedy@ and add tests. Created 4 years 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 11ae958f7a0f39ca8f319a4afb6316f184005c50..abe3394f6a8afc30f06721a03b17dcad34bd787c 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -64358,6 +64358,29 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram
+ name="SubresourceFilter.PageLoad.SubresourceEvaluation.TotalCPUDuration"
+ units="microseconds">
+ <owner>pkalinnikov@chromium.org</owner>
+ <summary>
+ Whenever a page load is finished with subresource filtering activated,
+ records the total thread CPU time spent on evaluating subresource requests,
+ aggregated across all frames.
+ </summary>
+</histogram>
+
+<histogram
+ name="SubresourceFilter.PageLoad.SubresourceEvaluation.TotalWallDuration"
+ units="microseconds">
+ <owner>pkalinnikov@chromium.org</owner>
+ <summary>
+ Whenever a page load is finished with subresource filtering activated,
+ records the total real time spent on evaluating subresource requests
engedy 2016/12/21 19:10:28 nit: Comma at end of line.
pkalinnikov 2016/12/22 16:53:04 Done.
+ aggregated across all frames. This includes the time spent on waiting or
+ being descheduled.
+ </summary>
+</histogram>
+
<histogram name="SubresourceFilter.Prompt.NumReloads" enum="BooleanRequested">
<owner>melandory@chromium.org</owner>
<summary>

Powered by Google App Engine
This is Rietveld 408576698