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

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

Issue 1921503003: service worker: Add UMA for start status grouped by start purpose. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rm extraneous file 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
« no previous file with comments | « content/browser/service_worker/service_worker_metrics.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 42aa0be767417fb91ad3a0eae0b44d7b7fcd87cb..42629e45642d0f4f83d6aad4488b0c03e9f5f662 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -47630,7 +47630,18 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<owner>falken@chromium.org</owner>
<summary>
The result of trying to start a Service Worker that is already installed.
- See also ServiceWorker.StartNewWorker.Status for new workers.
+ See also ServiceWorker.StartNewWorker.Status for new workers. See also
+ ServiceWorker.StartWorker.StatusByPurpose_* for the breakdown by event type.
+ </summary>
+</histogram>
+
+<histogram name="ServiceWorker.StartWorker.StatusByPurpose"
+ enum="ServiceWorkerStatusCode">
+ <owner>falken@chromium.org</owner>
+ <summary>
+ The result of trying to start a service worker for a particular event type
+ (see ServiceWorker.StartWorker.Status for the total). Only recorded for
+ installed workers. For event type:
</summary>
</histogram>
@@ -83525,12 +83536,12 @@ To add a new entry, add it with any value and run test to compute valid value.
<enum name="ServiceWorkerMetrics.EventType" type="int">
<int value="0" label="ACTIVATE"/>
<int value="1" label="INSTALL"/>
- <int value="2" label="FETCH (deprecated)"/>
+ <int value="2" label="FETCH (unused)"/>
<int value="3" label="SYNC"/>
<int value="4" label="NOTIFICATION_CLICK"/>
<int value="5" label="PUSH"/>
<int value="6" label="GEOFENCING"/>
- <int value="7" label="SERVICE_PORT_CONNECT"/>
+ <int value="7" label="SERVICE_PORT_CONNECT (unused)"/>
<int value="8" label="MESSAGE"/>
<int value="9" label="NOTIFICATION_CLOSE"/>
<int value="10" label="FETCH_MAIN_FRAME"/>
@@ -93069,6 +93080,24 @@ To add a new entry, add it with any value and run test to compute valid value.
<affected-histogram name="interstitial.interaction"/>
</histogram_suffixes>
+<histogram_suffixes name="ServiceWorker.EventType">
+ <suffix name="ACTIVATE" label="ACTIVATE"/>
+ <suffix name="INSTALL" label="INSTALL"/>
+ <suffix name="SYNC" label="SYNC"/>
+ <suffix name="NOTIFICATION_CLICK" label="NOTIFICATION_CLICK"/>
+ <suffix name="PUSH" label="PUSH"/>
+ <suffix name="GEOFENCING" label="GEOFENCING"/>
+ <suffix name="MESSAGE" label="MESSAGE"/>
+ <suffix name="NOTIFICATION_CLOSE" label="NOTIFICATION_CLOSE"/>
+ <suffix name="FETCH_MAIN_FRAME" label="FETCH_MAIN_FRAME"/>
+ <suffix name="FETCH_SUB_FRAME" label="FETCH_SUB_FRAME"/>
+ <suffix name="FETCH_SHARED_WORKER" label="FETCH_SHARED_WORKER"/>
+ <suffix name="FETCH_SUB_RESOURCE" label="FETCH_SUB_RESOURCE"/>
+ <suffix name="UNKNOWN" label="UNKNOWN"/>
+ <suffix name="FOREIGN_FETCH" label="FOREIGN_FETCH"/>
+ <affected-histogram name="ServiceWorker.StartWorker.StatusByPurpose"/>
+</histogram_suffixes>
+
<histogram_suffixes name="ServiceWorker.FailureStreak">
<affected-histogram name="ServiceWorker.StartWorker.AfterFailureStreak"/>
<suffix name="1" label="Failed one time."/>
« no previous file with comments | « content/browser/service_worker/service_worker_metrics.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698