Histogram appcache task queue and run times.

webkit/appcache/appcache_histograms.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/appcache/appcache_histograms.h"
 
 #include "base/metrics/histogram.h"
 
 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(
        "appcache.CheckResponseResult",
        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