Translate: add UMA to measure duration time of infobar user action

chrome/renderer/translate/translate_helper_metrics.cc

 // Copyright 2013 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 "chrome/renderer/translate/translate_helper_metrics.h"
 
 #include "base/metrics/histogram.h"
 
 namespace {
 
 // Constant string values to indicate UMA names. All entry should have
 // a corresponding index in MetricsNameIndex and an entry in |kMetricsEntries|.
 const char kRenderer4LanguageDetection[] = "Renderer4.LanguageDetection";
 const char kTranslateContentLanguage[] = "Translate.ContentLanguage";
 const char kTranslateLanguageVerification[] = "Translate.LanguageVerification";
 const char kTranslateTimeToBeReady[] = "Translate.TimeToBeReady";
 const char kTranslateTimeToLoad[] = "Translate.TimeToLoad";
 const char kTranslateTimeToTranslate[] = "Translate.TimeToTranslate";
+const char kTranslateUserActionDuration[] = "Translate.UserActionDuration";
 
 struct MetricsEntry {
   TranslateHelperMetrics::MetricsNameIndex index;
   const char* const name;
 };
 
 // This entry table should be updated when new UMA item is added.
 const MetricsEntry kMetricsEntries[] = {
   { TranslateHelperMetrics::UMA_LANGUAGE_DETECTION,
     kRenderer4LanguageDetection },
   { TranslateHelperMetrics::UMA_CONTENT_LANGUAGE,
     kTranslateContentLanguage },
   { TranslateHelperMetrics::UMA_LANGUAGE_VERIFICATION,
     kTranslateLanguageVerification },
   { TranslateHelperMetrics::UMA_TIME_TO_BE_READY,
     kTranslateTimeToBeReady },
   { TranslateHelperMetrics::UMA_TIME_TO_LOAD,
     kTranslateTimeToLoad },
   { TranslateHelperMetrics::UMA_TIME_TO_TRANSLATE,
     kTranslateTimeToTranslate },
+  { TranslateHelperMetrics::UMA_USER_ACTION_DURATION,
+    kTranslateUserActionDuration },
 };
 

[MAD, 2013/05/16 15:09:15]

While you're there, we could add a COMPILE_ASSERT here too...

[Takashi Toyoshima, 2013/05/16 16:46:45]

Done by another CL which is already landed :)

 }  // namespace
 
 namespace TranslateHelperMetrics {
 
 void ReportContentLanguage(const std::string& provided_code,
                            const std::string& revised_code) {
   if (provided_code.empty()) {
     UMA_HISTOGRAM_ENUMERATION(kTranslateContentLanguage,
                               CONTENT_LANGUAGE_NOT_PROVIDED,
                               CONTENT_LANGUAGE_MAX);
@@ skipping 25 matching lines @@
       kTranslateTimeToLoad,
       base::TimeDelta::FromMicroseconds(time_in_msec * 1000.0));
 }
 
 void ReportTimeToTranslate(double time_in_msec) {
   UMA_HISTOGRAM_MEDIUM_TIMES(
       kTranslateTimeToTranslate,
       base::TimeDelta::FromMicroseconds(time_in_msec * 1000.0));
 }
 
+void ReportUserActionDuration(base::TimeTicks begin, base::TimeTicks end) {
+  UMA_HISTOGRAM_LONG_TIMES(kTranslateUserActionDuration, end - begin);
+}
+
 #if defined(ENABLE_LANGUAGE_DETECTION)
 
 void ReportLanguageDetectionTime(base::TimeTicks begin, base::TimeTicks end) {
   UMA_HISTOGRAM_MEDIUM_TIMES(kRenderer4LanguageDetection, end - begin);
 }
 
 #endif  // defined(ENABLE_LANGUAGE_DETECTION)
 
 const char* GetMetricsName(MetricsNameIndex index) {
   for (size_t i = 0; i < arraysize(kMetricsEntries); ++i) {
     if (kMetricsEntries[i].index == index)
       return kMetricsEntries[i].name;
   }
   NOTREACHED();
   return NULL;
 }
 
 } // namespace TranslateHelperMetrics