Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1975)

Unified Diff: chrome/browser/autofill/personal_data_manager.cc

Issue 12282004: Added personal_data_manager android implementation for auto-populating auto-fill on android builds … (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autofill/personal_data_manager.cc
diff --git a/chrome/browser/autofill/personal_data_manager.cc b/chrome/browser/autofill/personal_data_manager.cc
index df49ded7237381f77f8561e0c597f52e14bb4024..831428929fa7eb80a22b6f0bb6036f351adf5187 100644
--- a/chrome/browser/autofill/personal_data_manager.cc
+++ b/chrome/browser/autofill/personal_data_manager.cc
@@ -869,9 +869,9 @@ void PersonalDataManager::LoadProfiles() {
pending_profiles_query_ = autofill_data->GetAutofillProfiles(this);
}
-// Win and Linux implementations do nothing. Mac implementation fills in the
-// contents of |auxiliary_profiles_|.
-#if !defined(OS_MACOSX)
+// Win and Linux implementations do nothing. Mac
aurimas (slooooooooow) 2013/02/15 19:08:35 Why break the line is such a weird place?
apiccion 2013/02/26 23:51:51 Done.
+// and Android implementation fills in the contents of |auxiliary_profiles_|.
Ilya Sherman 2013/02/16 04:14:37 nit: "implementation fills" -> "implementations fi
David Trainor- moved to gerrit 2013/02/19 19:03:13 implementations fill?
apiccion 2013/02/26 23:51:51 Done.
+#if !defined(OS_MACOSX) && !defined(OS_ANDROID)
void PersonalDataManager::LoadAuxiliaryProfiles() {
}
#endif

Powered by Google App Engine
This is Rietveld 408576698