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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 782903002: Breakdown ClientLoadedTime histograms into buckets based on number of clients. Fix RequestTimeThrot… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 c4f6eaced25000b3fb4828196b19c157c4d67530..4f8a980105150a6f60bd86528d074d186a460dad 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -29364,7 +29364,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
-<histogram name="ResourceScheduler.RequestTimeThrottled_Active">
Ilya Sherman 2014/12/09 02:59:36 Do these previous names already have a significant
aiolos (Not reviewing) 2015/01/05 23:37:59 Nope, they don't.
+<histogram name="ResourceScheduler.RequestTimeThrottled.Active">
<owner>aiolos@chromium.org</owner>
<summary>
The amount of time between when the request was created and when the
@@ -29373,7 +29373,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
-<histogram name="ResourceScheduler.RequestTimeThrottled_Background">
+<histogram name="ResourceScheduler.RequestTimeThrottled.Background">
<owner>aiolos@chromium.org</owner>
<summary>
The amount of time between when the request was created and when the
@@ -29382,7 +29382,7 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
-<histogram name="ResourceScheduler.RequestTimeThrottled_Other">
+<histogram name="ResourceScheduler.RequestTimeThrottled.Other">
<owner>aiolos@chromium.org</owner>
<summary>
The amount of time between when the request was created and when the
@@ -60852,6 +60852,23 @@ To add a new entry, add it with any value and run test to compute valid value.
name="ResourcePrefetchPredictor.Url.PredictedPrefetchMisses"/>
</histogram_suffixes>
+<histogram_suffixes name="ResourceSchedulerClientBreakDown">
+ <suffix name="1Client" label="One Client in the ResourceScheduler."/>
+ <suffix name="Max5Clients"
+ label="Two to five Clients in the ResourceScheduler."/>
+ <suffix name="Max15Clients"
+ label="Six to fifteen Clients in the ResourceScheduler."/>
+ <suffix name="Max30Clients"
+ label="Sixteen to thirty Clients in the ResourceScheduler."/>
+ <suffix name="Over30Clients"
+ label="Over thirty Clients in the ResourceScheduler."/>
+ <affected-histogram name="ResourceScheduler.ClientLoadedTime.Active"/>
+ <affected-histogram name="ResourceScheduler.ClientLoadedTime.Background"/>
+ <affected-histogram name="ResourceScheduler.ClientLoadedTime.Other"/>
+ <affected-histogram
+ name="ResourceScheduler.ClientLoadedTime.Other.SwitchedToActive"/>
+</histogram_suffixes>
+
<histogram_suffixes name="SBInterstitial">
<obsolete>
deprecated November 10 2012 crrev.com/167056
« content/browser/loader/resource_scheduler.cc ('K') | « content/browser/loader/resource_scheduler.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698