Mac: Autofill should not immediately request access to address book.

components/autofill/core/browser/autofill_manager.cc

Index: components/autofill/core/browser/autofill_manager.cc
diff --git a/components/autofill/core/browser/autofill_manager.cc b/components/autofill/core/browser/autofill_manager.cc
index 4c42479084aa4732c13e1457d04d75277419bf95..a8e7aee07623e45adb5ae2b3f1167a1962ea4c16 100644
--- a/components/autofill/core/browser/autofill_manager.cc
+++ b/components/autofill/core/browser/autofill_manager.cc
@@ -240,6 +240,23 @@ void AutofillManager::ShowAutofillSettings() {
   manager_delegate_->ShowAutofillSettings();
 }
 
+#if defined(OS_MACOSX)
+bool AutofillManager::ShouldShowAccessAddressBookSuggestion(
+    const FormData& data,
+    const FormFieldData& field_data) {
+  if (!personal_data_)
+    return false;
+  return personal_data_->ShouldShowAccessAddressBookSuggestion(
+      data, field_data, form_structures_);

[Ilya Sherman, 2014/05/22 15:13:59]

You should probably call GetCachedFormAndField(), and just pass in the field
type to this method.

[erikchen, 2014/05/22 20:40:59]

ah, good point. done.

+}
+
+bool AutofillManager::AccessAddressBook() {
+  if (!personal_data_)
+    return false;
+  return personal_data_->AccessAddressBook();
+}
+#endif  // defined(OS_MACOSX)
+
 bool AutofillManager::OnFormSubmitted(const FormData& form,
                                       const TimeTicks& timestamp) {
   if (!IsValidFormData(form))