[Autofill] Add UMA timing metrics for requestAutocomplete dialog.

chrome/browser/autofill/autofill_manager_delegate.h

Index: chrome/browser/autofill/autofill_manager_delegate.h
diff --git a/chrome/browser/autofill/autofill_manager_delegate.h b/chrome/browser/autofill/autofill_manager_delegate.h
index 019f36d53d14a6457a636cc6c0ced199cc9694d8..7ef1e76365aa6983432e1d9d76d8c70e50002c6c 100644
--- a/chrome/browser/autofill/autofill_manager_delegate.h
+++ b/chrome/browser/autofill/autofill_manager_delegate.h
@@ -21,6 +21,7 @@ namespace gfx {
 class Rect;
 }
 
+class AutofillMetrics;
 class FormStructure;
 class GURL;
 class InfoBarService;
@@ -32,6 +33,13 @@ struct FormData;
 
 namespace autofill {
 
+enum AutocompleteDialogRequester {
+  // Requested by the Autocheckout feature.
+  AUTOCOMPLETE_DIALOG_REQUESTER_AUTOCHECKOUT,
+  // Requested by the requestAutocomplete feature.
+  AUTOCOMPLETE_DIALOG_REQUESTER_REQUEST_AUTOCOMPLETE,
+};

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

These names are rather ickily long.  Should I drop the "autocomplete" prefix? 
Any suggestions for even better/shorter names?

[Dan Beam, 2013/02/01 17:17:28]

I think just REQUESTER_ is enough, that's what I did in my patch.

[Ilya Sherman, 2013/02/02 01:22:13]

Compromised by going with DIALOG_REQUESTER_, which seems to look ok in the
contexts where it's used.

+
 // A delegate interface that needs to be supplied to AutofillManager
 // by the embedder.
 //
@@ -83,6 +91,8 @@ class AutofillManagerDelegate {
       const FormData& form,
       const GURL& source_url,
       const content::SSLStatus& ssl_status,
+      const AutofillMetrics& metric_logger,
+      AutocompleteDialogRequester requester,
       const base::Callback<void(const FormStructure*)>& callback) = 0;
 
   // Called when the dialog for request autocomplete closes.