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

components/autofill/core/browser/personal_data_manager.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_PERSONAL_DATA_MANAGER_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_
 
 #include <set>
 #include <vector>
 
@@ skipping 182 matching lines @@
   // be associated with the user. More concretely, it checks if there are any
   // addresses with this country or if the user's system timezone is in the
   // given country.
   virtual bool IsCountryOfInterest(const std::string& country_code) const;
 
   // Returns our best guess for the country a user is likely to use when
   // inputting a new address. The value is calculated once and cached, so it
   // will only update when Chrome is restarted.
   virtual const std::string& GetDefaultCountryCodeForNewAddress() const;
 
+#if defined(OS_MACOSX)
+  // Whether Chromium has prompted the user for permission to access the user's

[Ilya Sherman, 2014/05/21 11:29:15]

nit: "Chromium" -> "Chrome" (in general, comments in the code refer to "Chrome"
(the main binary that we ship), even though the open source project name is
Chromium).

[erikchen, 2014/05/21 22:00:54]

good to know, thanks!

+  // address book.
+  bool HasPromptedForAccessToAddressBook();

[Ilya Sherman, 2014/05/21 11:29:15]

nit: Please leave a blank line after this one.

[erikchen, 2014/05/21 22:00:54]

Done.

+  // If Chromium has not prompted for access to the user's address book, prompts

[Ilya Sherman, 2014/05/21 11:29:15]

nit: Ditto.

[erikchen, 2014/05/21 22:00:54]

Done. I made sure that the diff file has no more references to 'Chromium'.

+  // the user for permission and blocks the process. Otherwise, this method has
+  // no effect.
+  void AccessAddressBook();
+#endif  // defined(OS_MACOSX)
+
  protected:
   // Only PersonalDataManagerFactory and certain tests can create instances of
   // PersonalDataManager.
   FRIEND_TEST_ALL_PREFIXES(AutofillMetricsTest, FirstMiddleLast);
   FRIEND_TEST_ALL_PREFIXES(AutofillMetricsTest, AutofillIsEnabledAtStartup);
   FRIEND_TEST_ALL_PREFIXES(PersonalDataManagerTest,
                            AggregateExistingAuxiliaryProfile);
   friend class autofill::AutofillInteractiveTest;
   friend class autofill::AutofillTest;
   friend class autofill::PersonalDataManagerFactory;
@@ skipping 131 matching lines @@
 
   // An observer to listen for changes to prefs::kAutofillEnabled.
   scoped_ptr<BooleanPrefMember> enabled_pref_;
 
   DISALLOW_COPY_AND_ASSIGN(PersonalDataManager);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_