mac: Don't show "Access Address Book" prompt if Autofill is disabled.

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 274 matching lines @@
 bool AutofillManager::ShouldShowAccessAddressBookSuggestion(
     const FormData& form,
     const FormFieldData& field) {
   if (!personal_data_)
     return false;
   FormStructure* form_structure = NULL;
   AutofillField* autofill_field = NULL;
   if (!GetCachedFormAndField(form, field, &form_structure, &autofill_field))
     return false;
 
+  if (!field.should_autocomplete)

[Evan Stade, 2014/08/28 17:18:58]

nit: combine this with the check on L288

[erikchen, 2014/08/28 17:28:46]

Done.

+    return false;
+
   return personal_data_->ShouldShowAccessAddressBookSuggestion(
       autofill_field->Type());
 }
 
 bool AutofillManager::AccessAddressBook() {
   if (!personal_data_)
     return false;
   return personal_data_->AccessAddressBook();
 }
 #endif  // defined(OS_MACOSX) && !defined(OS_IOS)
@@ skipping 916 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