Histogram appcache task queue and run times.

webkit/appcache/appcache_histograms.cc

Index: webkit/appcache/appcache_histograms.cc
===================================================================
--- webkit/appcache/appcache_histograms.cc	(revision 110360)
+++ webkit/appcache/appcache_histograms.cc	(working copy)
@@ -8,14 +8,12 @@
 
 namespace appcache {
 
-// static
 void AppCacheHistograms::CountInitResult(InitResultType init_result) {
   UMA_HISTOGRAM_ENUMERATION(
        "appcache.InitResult",
        init_result, NUM_INIT_RESULT_TYPES);
 }
 
-// static

[rvargas (doing something else), 2011/11/17 03:42:05]

nit: I'd prefer to keep the comment. I don't know if we have this written down
somewhere, but that's the pattern that I've always seen.

[michaeln, 2011/11/17 03:59:47]

Done.

 void AppCacheHistograms::CountCheckResponseResult(
     CheckResponseResultType result) {
   UMA_HISTOGRAM_ENUMERATION(
@@ -23,4 +21,19 @@
        result, NUM_CHECK_RESPONSE_RESULT_TYPES);
 }
 
+void AppCacheHistograms::AddTaskQueueTimeSample(
+    const base::TimeDelta& duration) {
+  UMA_HISTOGRAM_TIMES("appcache.TaskQueueTime", duration);
+}
+
+void AppCacheHistograms::AddTaskRunTimeSample(
+    const base::TimeDelta& duration) {
+  UMA_HISTOGRAM_TIMES("appcache.TaskRunTime", duration);
+}
+
+void AppCacheHistograms::AddTaskCompletionTimeSample(
+      const base::TimeDelta& duration) {
+  UMA_HISTOGRAM_TIMES("appcache.TaskCompletionTime", duration);
+}
+
 }  // namespace appcache