Recording in UMA when a user interacted with an address form or a credit card form.

components/autofill/core/browser/autofill_metrics.h

 // 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.
 
 #ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_METRICS_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_METRICS_H_
 
 #include <stddef.h>
 #include <string>
 
@@ skipping 225 matching lines @@
     USER_DID_AUTOFILL,
     // Same as above, but only logged once per page load.
     USER_DID_AUTOFILL_ONCE,
     // User edited a previously autofilled field.
     USER_DID_EDIT_AUTOFILLED_FIELD,
     // Same as above, but only logged once per page load.
     USER_DID_EDIT_AUTOFILLED_FIELD_ONCE,
     NUM_USER_HAPPINESS_METRICS,
   };
 
+  // Form Events for autofill.
+  // These events are triggered separetly for address and credit card forms.
+  enum AutofillFormEvent {

[Ilya Sherman, 2015/02/02 22:32:16]

Since this enum is already in a class that has "Autofill" in its name, which
furthermore lives in the "autofill" namespace, I'd drop the "Autofill" from this
name.

[Walter Cacau, 2015/02/03 00:47:40]

Done.

+    // User interacted with a field of this kind of form. Logged only once per
+    // page load.
+    INTERACTED_ONCE_FORM_EVENT = 0,

[Ilya Sherman, 2015/02/02 22:32:16]

nit: By convention, FORM_EVENT should be the prefix rather than the suffix.

[Walter Cacau, 2015/02/03 00:47:40]

Done.

+    // TODO(waltercacau): Remove the 2 here once we add more events.
+    //   This circunvents an assertion that gets triggered when you have

[Ilya Sherman, 2015/02/02 22:32:16]

nit: "circunvents" -> "circumvents"

[Walter Cacau, 2015/02/03 00:47:40]

Done.

+    //   only one bucket.
+    NUM_AUTOFILL_FORM_EVENTS = 2,
+  };
+
   // For measuring the network request time of various Wallet API calls. See
   // WalletClient::RequestType.
   enum WalletApiCallMetric {
     UNKNOWN_API_CALL,  // Catch all. Should never be used.
     ACCEPT_LEGAL_DOCUMENTS,
     AUTHENTICATE_INSTRUMENT,
     GET_FULL_WALLET,
     GET_WALLET_ITEMS,
     SAVE_TO_WALLET,
     NUM_WALLET_API_CALLS
@@ skipping 152 matching lines @@
   // This should be called each time a new profile is launched.
   static void LogIsAutofillEnabledAtStartup(bool enabled);
 
   // This should be called each time a new profile is launched.
   static void LogStoredProfileCount(size_t num_profiles);
 
   // Log the number of Autofill suggestions presented to the user when filling a
   // form.
   static void LogAddressSuggestionsCount(size_t num_suggestions);
 
+  // Utility to autofill form events in the relevant histograms depending on
+  // the presence of server and/or local data.
+  class FormEventLogger {
+   public:
+    FormEventLogger(bool is_credit_card);
+
+    // Sets if server data is available or not. If not called assumed false.
+    void set_is_server_data_available(bool is_server_data_available);

[Evan Stade, 2015/02/02 23:13:12]

inline this function

[Walter Cacau, 2015/02/03 00:47:40]

Done.

+
+    // Sets if server data is available or not. If not called assumed false.
+    void set_is_local_data_available(bool is_local_data_available);

[Evan Stade, 2015/02/02 23:13:12]

inline this function

[Walter Cacau, 2015/02/03 00:47:40]

Done.

+
+    // Should be called when the user interacted with an autofillable form.
+    void OnDidInteractWithAutofillableForm();

[Evan Stade, 2015/02/02 23:13:12]

move above the setters

[Walter Cacau, 2015/02/03 00:47:40]

Done.

+
+   private:
+    // Logs a form event.
+    void Log(AutofillFormEvent event) const;
+
+    bool is_credit_card_;

[Evan Stade, 2015/02/02 23:13:12]

nit: variable name doesn't sound quite right. It's more like whether this is a
FormEventLogger for a credit card fill event. So for_credit_card_ would make
more sense?

[Walter Cacau, 2015/02/03 00:47:40]

You are right. Doing is_for_credit_card_ just because we keep the is_ which
makes it more explicit that it is a boolean.

+    bool is_server_data_available_;
+    bool is_local_data_available_;
+    bool has_logged_interacted_;
+  };
+
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(AutofillMetrics);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_METRICS_H_