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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 2091353002: Generalize the reload PLMO to support other transition types. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add abort unit tests Created 4 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 5e0ad4ea615425c4fac55403586d13378614c966..2a534db847570d6d43ca1c847cc231303db51b87 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -97010,18 +97010,38 @@ To add a new entry, add it with any value and run test to compute valid value.
<histogram_suffixes name="PageLoadMetricsClients.Reload" separator="."
ordering="prefix">
+ <obsolete>
+ Deprecated in favor of LoadType.Reload.
+ </obsolete>
<suffix name="Clients.Reload"
label="PageLoadMetrics from a page that is reloaded"/>
<affected-histogram
name="PageLoad.PaintTiming.NavigationToFirstContentfulPaint"/>
</histogram_suffixes>
-<histogram_suffixes name="PageLoadMetricsClients.Reload.UserGesture"
- separator=".">
+<histogram_suffixes name="PageLoadMetricsLoadType" separator=".">
+ <suffix name="LoadType.Reload" label="Restricted to reloaded pages."/>
+ <suffix name="LoadType.ForwardBackNavigation"
+ label="Restricted to forward/back navigations."/>
+ <suffix name="LoadType.NewNavigation"
+ label="Restricted to new navigations (link clicks, URLs typed into the
+ URL box, etc)."/>
+ <affected-histogram
+ name="PageLoad.PaintTiming.NavigationToFirstContentfulPaint"/>
+</histogram_suffixes>
+
+<histogram_suffixes name="PageLoadMetricsUserGesture" separator=".">
<suffix name="UserGesture"
- label="PageLoadMetrics from a page that is reloaded by a user gesture"/>
+ label="Restricted to pages loaded via a user gesture."/>
+ <affected-histogram
+ name="PageLoad.Clients.Reload.PaintTiming.NavigationToFirstContentfulPaint">
+ <obsolete>
+ Deprecated in favor of
+ PageLoad.PaintTiming.NavigationToFirstContentfulPaint.LoadType.Reload.
+ </obsolete>
+ </affected-histogram>
<affected-histogram
- name="PageLoad.Clients.Reload.PaintTiming.NavigationToFirstContentfulPaint"/>
+ name="PageLoad.PaintTiming.NavigationToFirstContentfulPaint.LoadType.Reload"/>
</histogram_suffixes>
<histogram_suffixes name="PageLoadType">

Powered by Google App Engine
This is Rietveld 408576698