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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 1871393002: Add NavigationToFirstContentfulPaint UMA for ServiceWorker controlled pages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: incorporated ksakamoto's comment Created 4 years, 8 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 e3407522646576ba021a2bfe8cf0746037e45e7a..8fbff7b0c1042c6b497ce7c94d3cd3f96999cb66 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -90700,6 +90700,12 @@ To add a new entry, add it with any value and run test to compute valid value.
<affected-histogram name="PageLoad.Timing2.NavigationToLoadEventFired"/>
</histogram_suffixes>
+<histogram_suffixes name="PageLoadServiceWorker" separator=".">
+ <suffix name="ServiceWorker"
+ label="The page was controlled by the Service Worker."/>
+ <affected-histogram name="PageLoad.Timing2.NavigationToFirstContentfulPaint"/>
+</histogram_suffixes>
+
<histogram_suffixes name="PageLoadType">
<suffix name="HistoryLoad"
label="but only for user pressing back or forward"/>

Powered by Google App Engine
This is Rietveld 408576698