ServiceWorker: Add UMA to track the execution time of onmessage event

BUG=407997 TEST=N/A Review URL: https://codereview.chromium.org/617263008 Cr-Commit-Position: refs/heads/master@{#297957}
marcosholgado · Oct 3, 2014 · 20894f5 · 20894f5
1 parent 05b6445
commit 20894f5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/content/renderer/service_worker/service_worker_script_context.cc b/content/renderer/service_worker/service_worker_script_context.cc
@@ -227,7 +227,13 @@ void ServiceWorkerScriptContext::OnPostMessage(
     }
   }
 
+  // dispatchMessageEvent is expected to execute onmessage function
+  // synchronously.
+  base::TimeTicks before = base::TimeTicks::Now();
   proxy_->dispatchMessageEvent(message, ports);
+  UMA_HISTOGRAM_TIMES(
+      "ServiceWorker.MessageEventExecutionTime",
+      base::TimeTicks::Now() - before);
 }
 
 void ServiceWorkerScriptContext::OnDidGetClientDocuments(

diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
@@ -29506,6 +29506,11 @@ Therefore, the affected-histogram name has to have at least one dot in it.
   <summary>Execution time of ServiceWorkerGlobalScope.oninstall.</summary>
 </histogram>
 
+<histogram name="ServiceWorker.MessageEventExecutionTime" units="millisecond">
+  <owner>shimazu@chromium.org</owner>
+  <summary>Execution time of ServiceWorkerGlobalScope.onmessage.</summary>
+</histogram>
+
 <histogram name="ServiceWorker.ScriptSize" units="bytes">
   <owner>ksakamoto@chromium.org</owner>
   <summary>