[Autofill] Add UMA timing metrics for requestAutocomplete dialog.

chrome/browser/autofill/autofill_metrics.cc

Index: chrome/browser/autofill/autofill_metrics.cc
diff --git a/chrome/browser/autofill/autofill_metrics.cc b/chrome/browser/autofill/autofill_metrics.cc
index 94f3cff695f4904f9ef45f506a6b87063ed14d3a..c01ca89e54f599785517f62e79938f139f4785b4 100644
--- a/chrome/browser/autofill/autofill_metrics.cc
+++ b/chrome/browser/autofill/autofill_metrics.cc
@@ -268,6 +268,25 @@ void AutofillMetrics::LogAutocheckoutInfoBarMetric(InfoBarMetric metric) const {
                             NUM_INFO_BAR_METRICS);
 }
 
+void AutofillMetrics::LogRequestAutocompleteUiDuration(
+    const base::TimeDelta& duration,
+    DialogDismissalAction dismissal_action) const {
+  UMA_HISTOGRAM_LONG_TIMES("RequestAutocomplete.UiDuration", duration);

[Albert Bodenhamer, 2013/01/31 16:31:06]

What's the best way to split this out by invocation type?  autocheckout will run
through this flow as well, but we'll want to track that separately.

[Ilya Sherman, 2013/02/01 09:55:17]

Done.

+
+  switch (dismissal_action) {
+    case DIALOG_ACCEPTED:
+      UMA_HISTOGRAM_LONG_TIMES("RequestAutocomplete.UiDuration.Submit",
+                               duration);
+      break;
+
+    case DIALOG_CANCELED:
+      UMA_HISTOGRAM_LONG_TIMES("RequestAutocomplete.UiDuration.Cancel",
+                               duration);
+      break;
+  }
+}
+
+
 void AutofillMetrics::LogDeveloperEngagementMetric(
     DeveloperEngagementMetric metric) const {
   DCHECK(metric < NUM_DEVELOPER_ENGAGEMENT_METRICS);