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

Unified Diff: components/autofill/browser/personal_data_manager.h

Issue 16034018: [Autofill] Support "importing" verified profiles and credit cards. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix up comments Created 7 years, 6 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: components/autofill/browser/personal_data_manager.h
diff --git a/components/autofill/browser/personal_data_manager.h b/components/autofill/browser/personal_data_manager.h
index 0457c20f5b85f08b52af0b5efb4f3e5f14506ea4..555e623a1e176936d85f1b41f530e761ebc21dd3 100644
--- a/components/autofill/browser/personal_data_manager.h
+++ b/components/autofill/browser/personal_data_manager.h
@@ -166,11 +166,11 @@ class PersonalDataManager : public WebDataServiceConsumer,
static bool IsValidLearnableProfile(const AutofillProfile& profile,
const std::string& app_locale);
- // Merges |profile| into one of the |existing_profiles| if possible; otherwise
- // appends |profile| to the end of that list. Fills |merged_profiles| with the
- // result.
+ // Merges |new_profile| into one of the |existing_profiles| if possible;
+ // otherwise appends |new_profile| to the end of that list. Fills
+ // |merged_profiles| with the result.
static bool MergeProfile(
- const AutofillProfile& profile,
+ const AutofillProfile& new_profile,
const std::vector<AutofillProfile*>& existing_profiles,
const std::string& app_locale,
std::vector<AutofillProfile>* merged_profiles);

Powered by Google App Engine
This is Rietveld 408576698