ServiceWorker: Keep the worker alive until FetchEvent.waitUntil settles

content/browser/service_worker/service_worker_metrics.h

Index: content/browser/service_worker/service_worker_metrics.h
diff --git a/content/browser/service_worker/service_worker_metrics.h b/content/browser/service_worker/service_worker_metrics.h
index 9be6cdb4a5147ff15ca6a9d2c40d4e7da42ee362..07ecbe40ced788b9db4685a1d171fad52801e4ba 100644
--- a/content/browser/service_worker/service_worker_metrics.h
+++ b/content/browser/service_worker/service_worker_metrics.h
@@ -80,6 +80,13 @@ class ServiceWorkerMetrics {
   };
 
   // Used for UMA. Append-only.
+  // This class is used to indicate which event is fired/finished. Most events
+  // have only one request that starts event and one response that finishes

[falken, 2016/06/22 05:53:33]

nit: s/starts event/starts the event and s/finishes event/finishes the event

[shimazu, 2016/06/22 07:13:39]

Done.

+  // event, but the fetch and the foreign fetch event have two responses, so
+  // there are two types of EventType to break down the measurement into two:
+  // FETCH/FOREIGN_FETCH and FETCH_WAITUNTIL/FOREIGN_FETCH_WAITUNTIL.
+  // Moreover, FETCH is separated into the four: MAIN_FRAME, SUB_FRAME,
+  // SHARED_WORKER and SUB_RESOURCE for more detailed UMA.
   enum class EventType {
     ACTIVATE = 0,
     INSTALL = 1,
@@ -97,6 +104,8 @@ class ServiceWorkerMetrics {
     FETCH_SUB_RESOURCE = 13,
     UNKNOWN = 14,  // Used when event type is not known.
     FOREIGN_FETCH = 15,
+    FETCH_WAITUNTIL = 16,
+    FOREIGN_FETCH_WAITUNTIL = 17,
     // Add new events to record here.
     NUM_TYPES
   };