Build Autofill component on iOS.

components/autofill/core/browser/personal_data_manager.cc

Index: components/autofill/core/browser/personal_data_manager.cc
diff --git a/components/autofill/core/browser/personal_data_manager.cc b/components/autofill/core/browser/personal_data_manager.cc
index 28d92d4350386db2a54d691ff3e847a830d29a45..48438a87a93bd06e74f92b033320bbfb80ae1b4a 100644
--- a/components/autofill/core/browser/personal_data_manager.cc
+++ b/components/autofill/core/browser/personal_data_manager.cc
@@ -866,9 +866,9 @@ void PersonalDataManager::LoadProfiles() {
   pending_profiles_query_ = database_->GetAutofillProfiles(this);
 }
 
-// Win and Linux implementations do nothing. Mac and Android implementations
-// fill in the contents of |auxiliary_profiles_|.
-#if !defined(OS_MACOSX) && !defined(OS_ANDROID)
+// Win, Linux, and iOS implementations do nothing. Mac and Android
+// implementations fill in the contents of |auxiliary_profiles_|.
+#if defined(OS_IOS) || (!defined(OS_MACOSX) && !defined(OS_ANDROID))

[Ilya Sherman, 2013/12/11 22:50:38]

Why are changes to this line needed?  Does OS_IOS also cause OS_MACOSX to be
defined?

[blundell, 2013/12/12 20:53:54]

OS_IOS does indeed cause OS_MACOSX to be defined (it's done this way because it
follows the way that Apple manages its defines).

On 2013/12/11 22:50:38, Ilya Sherman wrote:

[Ilya Sherman, 2013/12/12 22:34:52]

Ok, good to know!  Might be worth adding a comment here to that effect, since
that's not obvious to most people not familiar with the OS defines.  Though,
maybe it's not worth it, because adding such comments everywhere where they are
relevant might result in the entire codebase littered with lots of repetitive
comments.  Up to your discretion, I guess.

On 2013/12/12 20:53:54, blundell wrote:

[blundell, 2013/12/12 22:41:29]

I don't think that we comment this in general.

On 2013/12/12 22:34:52, Ilya Sherman wrote:

 void PersonalDataManager::LoadAuxiliaryProfiles() const {
 }
 #endif