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

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

Issue 2362743002: Add TaskScheduler.NumTasksBetweenWaits.[pool] histogram. (Closed)
Patch Set: CR gab/robliao #7-10 Created 4 years, 3 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 4fd1a7941c3d029237eacdd0f14768df4234c2a4..85da32be5e18c0fd1f98513248561a1a31dbbff1 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -62710,6 +62710,17 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="TaskScheduler.NumTasksBetweenWaits" units="tasks">
+ <owner>fdoray@chromium.org</owner>
+ <owner>gab@chromium.org</owner>
+ <owner>robliao@chromium.org</owner>
+ <summary>
+ Number of tasks executed by a SchedulerWorker between two waits on its
+ WaitableEvent. This should be maximized without affecting perceived browser
+ performance.
+ </summary>
+</histogram>
+
<histogram name="TaskScheduler.TaskLatency" units="ms">
<owner>fdoray@chromium.org</owner>
<owner>gab@chromium.org</owner>
@@ -107135,6 +107146,7 @@ To add a new entry, add it with any value and run test to compute valid value.
label="Applies to the ForegroundFileIO worker pool."/>
<suffix name="ForegroundPool" label="Applies to the Foreground worker pool."/>
<affected-histogram name="TaskScheduler.DetachDuration"/>
+ <affected-histogram name="TaskScheduler.NumTasksBetweenWaits"/>
<affected-histogram name="TaskScheduler.TaskLatency"/>
</histogram_suffixes>

Powered by Google App Engine
This is Rietveld 408576698