tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 79775fa98721d822d8563298e17b80a0b30d13e5..fed5c7eec1481d0850b2b2ac5d0f1c5d375bf5d4 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -33448,6 +33448,30 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   </summary>
 </histogram>
 
+<histogram name="ServiceWorkerVersion.StartWorkerStatus"

[Alexei Svitkine (slow), 2015/03/04 15:46:38]

Nit: Could you avoid introducing a new top-level prefix?

Since there's already a ServiceWorker.* prefix, I suggest changing these to
ServiceWorker.Version.*.

Although it's also not clear to me what ServiceWorkerVersion.StartWorkerStatus
has to do with the service worker version. Should it just be
ServiceWorker.StartWorkerStatus or actually even better
ServiceWorker.StartWorker.Status? Same for the others.

[falken, 2015/03/05 08:33:11]

Ah I didn't realize a new top-level prefix was a big deal, I just took the class
name. ServiceWorker.StartWorker.*" sgtm. Done.

+    units="ServiceWorkerStatusCode">
+  <owner>falken@chromium.org</owner>
+  <summary>The result of trying to start a Service Worker.</summary>
+</histogram>
+
+<histogram name="ServiceWorkerVersion.StartWorkerTime" units="milliseconds">
+  <owner>falken@chromium.org</owner>
+  <summary>
+    The time taken to start a Service Worker end-to-end, from process allocation
+    to ACK of started from the renderer. This possibly includes script download
+    time, but the metric is not recorded if DevTools was ever attached to the
+    Service Worker during startup.
+  </summary>
+</histogram>
+
+<histogram name="ServiceWorkerVersion.StartWorkerTimeoutPhase"
+    units="EmbeddedWorkerStartingPhase">
+  <owner>falken@chromium.org</owner>
+  <summary>
+    The phase the EmbeddedWorker was in when ServiceWorker startup timed out.
+  </summary>
+</histogram>
+
 <histogram name="SessionCrashed.Bubble" enum="SessionCrashedBubbleUserAction">
   <owner>yiyaoliu@chromium.org</owner>
   <summary>How did the user interact with the SessionCrashed Bubble?</summary>
@@ -46671,6 +46695,16 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   <int value="1" label="Screen unlocked (via EasyUnlock)"/>
 </enum>
 
+<enum name="EmbeddedWorkerStartingPhase" type="int">
+  <int value="0" label="NOT_STARTING"/>
+  <int value="1" label="ALLOCATING_PROCESS"/>
+  <int value="2" label="REGISTERING_TO_DEVTOOLS"/>
+  <int value="3" label="SENT_START_WORKER"/>
+  <int value="4" label="SCRIPT_DOWNLOADING"/>
+  <int value="5" label="SCRIPT_LOADED"/>
+  <int value="6" label="SCRIPT_EVALUATED"/>
+</enum>
+
 <enum name="EnterpriseCheckError" type="int">
   <summary>
     Defined as DomainCheckErrors in
@@ -58212,6 +58246,24 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="2" label="Read data error"/>
 </enum>
 
+<enum name="ServiceWorkerStatusCode" type="int">
+  <int value="0" label="SERVICE_WORKER_OK"/>
+  <int value="1" label="SERVICE_WORKER_ERROR_FAILED"/>
+  <int value="2" label="SERVICE_WORKER_ERROR_ABORT"/>
+  <int value="3" label="SERVICE_WORKER_ERROR_START_WORKER_FAILED"/>
+  <int value="4" label="SERVICE_WORKER_ERROR_PROCESS_NOT_FOUND"/>
+  <int value="5" label="SERVICE_WORKER_ERROR_NOT_FOUND"/>
+  <int value="6" label="SERVICE_WORKER_ERROR_EXISTS"/>
+  <int value="7" label="SERVICE_WORKER_ERROR_INSTALL_WORKER_FAILED"/>
+  <int value="8" label="SERVICE_WORKER_ERROR_ACTIVATE_WORKER_FAILED"/>
+  <int value="9" label="SERVICE_WORKER_ERROR_IPC_FAILED"/>
+  <int value="10" label="SERVICE_WORKER_ERROR_NETWORK"/>
+  <int value="11" label="SERVICE_WORKER_ERROR_SECURITY"/>
+  <int value="12" label="SERVICE_WORKER_ERROR_EVENT_WAITUNTIL_REJECTED"/>
+  <int value="13" label="SERVICE_WORKER_ERROR_STATE"/>
+  <int value="14" label="SERVICE_WORKER_ERROR_TIMEOUT"/>
+</enum>
+
 <enum name="ServiceWorkerWriteResponseResult" type="int">
   <int value="0" label="OK"/>
   <int value="1" label="Write headers error"/>