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

components/autofill/core/browser/autofill_external_delegate.h

 // 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.
 
 #ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_EXTERNAL_DELEGATE_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_EXTERNAL_DELEGATE_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 124 matching lines @@
   bool has_suggestion_;
 
   // Have we already shown Autofill suggestions for the field the user is
   // currently editing?  Used to keep track of state for metrics logging.
   bool has_shown_popup_for_current_edit_;
 
   // The current data list values.
   std::vector<base::string16> data_list_values_;
   std::vector<base::string16> data_list_labels_;
 
+#if defined(OS_MACOSX)
+  // If this property is set, then the autofill popup should be immediately
+  // redisplayed when the text field loses focus.
+  bool redisplay_autofill_;

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

nit: Perhaps "redisplay_popop_on_focus_lost_" or something like that?  I'd
definitely replace "autofill" with "popup"; the rest I feel less strongly about.

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

I've changed the property name to your suggestion.

+#endif
+
   base::WeakPtrFactory<AutofillExternalDelegate> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(AutofillExternalDelegate);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_AUTOFILL_EXTERNAL_DELEGATE_H_