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

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

Side-by-side diff isn't available for this file because of its large size.
Issue 2034663002: ServiceWorker: Keep the worker alive until FetchEvent.waitUntil settles (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Separate DispathEvent into two part 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 e27be831c1d97deee5003d779751ae1067c2dc53..12e838032a62fbf83b5314bfc3cf3408482f93d0 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -49463,8 +49463,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<owner>jeremyarcher@chromium.org</owner>
<summary>
The time taken between dispatching a FetchEvent to a Service Worker and
- receiving a fallback-to-network reply. Includes the time for the waitUntil()
- promise to settle, if any.
+ receiving a fallback-to-network reply.
</summary>
</histogram>
@@ -49508,6 +49507,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="ServiceWorker.FetchEvent.WaitUntil.Time" units="ms">
+ <owner>shimazu@chromium.org</owner>
+ <summary>
+ The time taken between dispatching a FetchEvent to a Service Worker and
+ finishing the FetchEvent. Includes the time for the waitUntil() promise to
+ settle. If there is no waitUntil promise, this will be almost the same with
+ HasResponse.Time or Fallback.Time.
+ </summary>
+</histogram>
+
<histogram name="ServiceWorker.FetchEventExecutionTime" units="ms">
<owner>shimazu@chromium.org</owner>
<obsolete>
@@ -86631,6 +86640,7 @@ To add a new entry, add it with any value and run test to compute valid value.
<int value="13" label="FETCH_SUB_RESOURCE"/>
<int value="14" label="UNKNOWN"/>
<int value="15" label="FOREIGN_FETCH"/>
+ <int value="16" label="FETCH_WAITUNTIL"/>
</enum>
<enum name="ServiceWorkerReadResponseResult" type="int">

Powered by Google App Engine
This is Rietveld 408576698