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

components/autofill/core/browser/autofill_manager.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 "components/autofill/core/browser/autofill_manager.h"
 
 #include <stddef.h>
 
 #include <limits>
 #include <map>
@@ skipping 131 matching lines @@
     AutofillDriver* driver,
     AutofillClient* client,
     const std::string& app_locale,
     AutofillDownloadManagerState enable_download_manager)
     : driver_(driver),
       client_(client),
       app_locale_(app_locale),
       personal_data_(client->GetPersonalDataManager()),
       autocomplete_history_manager_(
           new AutocompleteHistoryManager(driver, client)),
+      address_form_event_logger_(
+          new AutofillMetrics::FormEventLogger(false /* is_credit_card */)),
+      credit_card_form_event_logger_(
+          new AutofillMetrics::FormEventLogger(true /* is_credit_card */)),
       has_logged_autofill_enabled_(false),
       has_logged_address_suggestions_count_(false),
       did_show_suggestions_(false),
       user_did_type_(false),
       user_did_autofill_(false),
       user_did_edit_autofilled_field_(false),
       external_delegate_(NULL),
       test_delegate_(NULL),
       weak_ptr_factory_(this) {
   if (enable_download_manager == ENABLE_AUTOFILL_DOWNLOAD_MANAGER) {
@@ skipping 286 matching lines @@
   if (!IsValidFormData(form) || !IsValidFormFieldData(field))
     return;
 
   std::vector<Suggestion> suggestions;
 
   external_delegate_->OnQuery(query_id,
                               form,
                               field,
                               bounding_box,
                               display_warning);
+
+  // Need to refresh models before using the form_event_loggers.
+  bool is_autofill_possible = RefreshDataModels();
+
   FormStructure* form_structure = NULL;
   AutofillField* autofill_field = NULL;
-  if (RefreshDataModels() &&
+  bool got_autofillable_form =
+      GetCachedFormAndField(form, field, &form_structure, &autofill_field) &&
+      // Don't send suggestions or track forms that aren't auto-fillable.
+      form_structure->IsAutofillable();
+
+  // Logging interactions of forms that are autofillable.
+  if (got_autofillable_form) {
+    AutofillType type = autofill_field->Type();
+    bool is_filling_credit_card = (type.group() == CREDIT_CARD);

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

nit:

  if (autofill_field->Type().group() == CREDIT_CARD)

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

Done.

+    if (is_filling_credit_card)
+      credit_card_form_event_logger_->OnDidInteractWithAutofillableForm();
+    else
+      address_form_event_logger_->OnDidInteractWithAutofillableForm();
+  }
+
+  if (is_autofill_possible &&
       driver_->RendererIsAvailable() &&
-      GetCachedFormAndField(form, field, &form_structure, &autofill_field) &&
-      // Don't send suggestions for forms that aren't auto-fillable.
-      form_structure->IsAutofillable()) {
+      got_autofillable_form) {
     AutofillType type = autofill_field->Type();
     bool is_filling_credit_card = (type.group() == CREDIT_CARD);
     if (is_filling_credit_card) {
       suggestions = GetCreditCardSuggestions(field, type);
     } else {
       suggestions =
           GetProfileSuggestions(*form_structure, field, *autofill_field);
     }
-
     if (!suggestions.empty()) {
       // Don't provide Autofill suggestions when Autofill is disabled, and don't
       // provide credit card suggestions for non-HTTPS pages. However, provide a
       // warning to the user in these cases.
       int warning = 0;
       if (is_filling_credit_card && !FormIsHTTPS(*form_structure)) {
         warning = IDS_AUTOFILL_WARNING_INSECURE_CONNECTION;
       }
       if (warning) {
         Suggestion warning_suggestion(l10n_util::GetStringUTF16(warning));
@@ skipping 336 matching lines @@
   if (!download_manager_)
     return false;
 
   return download_manager_->StartUploadRequest(form_structure,
                                                false /* was_autofilled */,
                                                available_field_types);
 }
 
 void AutofillManager::Reset() {
   form_structures_.clear();
+  address_form_event_logger_.reset(
+      new AutofillMetrics::FormEventLogger(false /* is_credit_card */));
+  credit_card_form_event_logger_.reset(
+      new AutofillMetrics::FormEventLogger(true /* is_credit_card */));
   has_logged_autofill_enabled_ = false;
   has_logged_address_suggestions_count_ = false;
   did_show_suggestions_ = false;
   user_did_type_ = false;
   user_did_autofill_ = false;
   user_did_edit_autofilled_field_ = false;
   unmasking_card_ = CreditCard();
   unmasking_query_id_ = -1;
   unmasking_form_ = FormData();
   unmasking_field_ = FormFieldData();
   forms_loaded_timestamps_.clear();
   initial_interaction_timestamp_ = TimeTicks();
   external_delegate_->Reset();
 }
 
 AutofillManager::AutofillManager(AutofillDriver* driver,
                                  AutofillClient* client,
                                  PersonalDataManager* personal_data)
     : driver_(driver),
       client_(client),
       app_locale_("en-US"),
       personal_data_(personal_data),
       autocomplete_history_manager_(
           new AutocompleteHistoryManager(driver, client)),
+      address_form_event_logger_(
+          new AutofillMetrics::FormEventLogger(false /* is_credit_card */)),
+      credit_card_form_event_logger_(
+          new AutofillMetrics::FormEventLogger(true /* is_credit_card */)),
       has_logged_autofill_enabled_(false),
       has_logged_address_suggestions_count_(false),
       did_show_suggestions_(false),
       user_did_type_(false),
       user_did_autofill_(false),
       user_did_edit_autofilled_field_(false),
       unmasking_query_id_(-1),
       external_delegate_(NULL),
       test_delegate_(NULL),
       weak_ptr_factory_(this) {
   DCHECK(driver_);
   DCHECK(client_);
 }
 
-bool AutofillManager::RefreshDataModels() const {
+bool AutofillManager::RefreshDataModels() {
   if (!IsAutofillEnabled())
     return false;
 
   // No autofill data to return if the profiles are empty.
-  if (personal_data_->GetProfiles().empty() &&
-      personal_data_->GetCreditCards().empty()) {
+  const std::vector<AutofillProfile*>& profiles =
+      personal_data_->GetProfiles();
+  const std::vector<CreditCard*>& credit_cards =
+      personal_data_->GetCreditCards();
+
+  // Updating the FormEventLoggers for addresses and credit cards.
+  {
+    bool is_server_data_available = false;
+    bool is_local_data_available = false;
+    for (CreditCard* credit_card : credit_cards) {
+      if (credit_card->record_type() == CreditCard::MASKED_SERVER_CARD ||
+          credit_card->record_type() == CreditCard::FULL_SERVER_CARD)

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

curlies required

but not if you make it shorter by just checking against LOCAL_CARD type

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

Done.

+        is_server_data_available = true;
+      else
+        is_local_data_available = true;
+    }
+    credit_card_form_event_logger_->set_is_server_data_available(
+        is_server_data_available);
+    credit_card_form_event_logger_->set_is_local_data_available(
+        is_local_data_available);
+  }
+  {
+    bool is_server_data_available = false;
+    bool is_local_data_available = false;
+    for (AutofillProfile* profile : profiles) {
+      if (profile->record_type() == AutofillProfile::SERVER_PROFILE)
+        is_server_data_available = true;
+      else
+        is_local_data_available = true;
+    }
+    address_form_event_logger_->set_is_server_data_available(
+        is_server_data_available);
+    address_form_event_logger_->set_is_local_data_available(
+        is_local_data_available);
+  }
+
+  if (profiles.empty() && credit_cards.empty()) {

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

nit: no curlies

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

Done.

     return false;
   }
 
   return true;
 }
 
 bool AutofillManager::GetProfileOrCreditCard(
     int unique_id,
     const AutofillDataModel** data_model,
     size_t* variant,
@@ skipping 440 matching lines @@
     return false;
 
   // Disregard forms that we wouldn't ever autofill in the first place.
   if (!form.ShouldBeParsed())
     return false;
 
   return true;
 }
 
 }  // namespace autofill