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

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 222 matching lines @@
   // work if the delegate has a pointer to the AutofillManager, but
   // future directions may not need such a pointer.
   external_delegate_ = delegate;
   autocomplete_history_manager_->SetExternalDelegate(delegate);
 }
 
 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))
     return false;
 
   // Let Autocomplete know as well.
   autocomplete_history_manager_->OnFormSubmitted(form);
 
   // Grab a copy of the form data.
   scoped_ptr<FormStructure> submitted_form(new FormStructure(form));
@@ skipping 885 matching lines @@
     return false;
 
   // Disregard forms that we wouldn't ever autofill in the first place.
   if (!form.ShouldBeParsed(true))
     return false;
 
   return true;
 }
 
 }  // namespace autofill