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

chrome/renderer/translate/translate_helper_metrics.cc

Index: chrome/renderer/translate/translate_helper_metrics.cc
diff --git a/chrome/renderer/translate/translate_helper_metrics.cc b/chrome/renderer/translate/translate_helper_metrics.cc
index a014da42470cee214231bc533a468e50829b93ef..c1f6100bd6519354b197255c549a465476a40d00 100644
--- a/chrome/renderer/translate/translate_helper_metrics.cc
+++ b/chrome/renderer/translate/translate_helper_metrics.cc
@@ -16,6 +16,7 @@ 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;
@@ -36,6 +37,8 @@ const MetricsEntry kMetricsEntries[] = {
     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
@@ -83,6 +86,10 @@ void ReportTimeToTranslate(double time_in_msec) {
       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) {