mac: Add histograms to measure impact of Address Book integration.

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>
 #include <set>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/guid.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/field_trial.h"
+#include "base/metrics/histogram.h"

[Ilya Sherman, 2015/05/15 21:14:58]

nit: Please include histogram_macros.h instead.

[erikchen, 2015/05/15 21:37:23]

Done.

 #include "base/prefs/pref_service.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "components/autofill/core/browser/autocomplete_history_manager.h"
 #include "components/autofill/core/browser/autofill_client.h"
 #include "components/autofill/core/browser/autofill_data_model.h"
 #include "components/autofill/core/browser/autofill_experiments.h"
 #include "components/autofill/core/browser/autofill_external_delegate.h"
@@ skipping 663 matching lines @@
 }
 
 void AutofillManager::OnHidePopup() {
   if (!IsAutofillEnabled())
     return;
 
   autocomplete_history_manager_->CancelPendingQuery();
   client_->HideAutofillPopup();
 }
 
+void AutofillManager::OnUserDidAcceptAutofillSuggestion(int unique_id) {
+#if defined(OS_MACOSX) && !defined(OS_IOS)
+  size_t variant = 0;
+  const AutofillProfile* profile = nullptr;
+  bool result = GetProfile(unique_id, &profile, &variant);
+  if (!result)
+    return;
+
+  bool is_from_address_book =
+      profile->record_type() == AutofillProfile::AUXILIARY_PROFILE;
+  UMA_HISTOGRAM_BOOLEAN("Autofill.MacAddressBook.AcceptedResultFromAddressBook",

[Ilya Sherman, 2015/05/15 21:14:58]

nit: I'd name this "AcceptedResultIsFromAddressBook", so it's clearer that this
is comparing AB vs. non-AB, as opposed to accepted vs. didn't accept.  Also, I'd
probably replace "Result" with "Suggestion" or "Profile".

[erikchen, 2015/05/15 21:37:23]

Done.

+                        is_from_address_book);
+#endif  // defined(OS_MACOSX) && !defined(OS_IOS)
+}
+
 bool AutofillManager::RemoveAutofillProfileOrCreditCard(int unique_id) {
   std::string guid;
   size_t variant = 0;
   const CreditCard* credit_card = nullptr;
   const AutofillProfile* profile = nullptr;
   if (GetCreditCard(unique_id, &credit_card)) {
     if (credit_card->record_type() != CreditCard::LOCAL_CARD)
       return false;
 
     guid = credit_card->guid();
@@ skipping 798 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