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

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

Issue 11360055: [Autofill] Rename GetInfo and SetInfo to GetRawInfo and SetRawInfo (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase harder Created 8 years, 1 month 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
« no previous file with comments | « chrome/browser/autofill/form_group.cc ('k') | chrome/browser/autofill/personal_data_manager_mac.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 f1f4610714a2668d41582d97ecb78f0c8a180727..8e7c62eca279cbece6e7574fdbc680e719400f4b 100644
--- a/chrome/browser/autofill/personal_data_manager.cc
+++ b/chrome/browser/autofill/personal_data_manager.cc
@@ -93,10 +93,10 @@ bool IsValidZip(const string16& value) {
// filled. No verification of validity of the contents is preformed. This is
// and existence check only.
bool IsMinimumAddress(const AutofillProfile& profile) {
- return !profile.GetInfo(ADDRESS_HOME_LINE1).empty() &&
- !profile.GetInfo(ADDRESS_HOME_CITY).empty() &&
- !profile.GetInfo(ADDRESS_HOME_STATE).empty() &&
- !profile.GetInfo(ADDRESS_HOME_ZIP).empty();
+ return !profile.GetRawInfo(ADDRESS_HOME_LINE1).empty() &&
+ !profile.GetRawInfo(ADDRESS_HOME_CITY).empty() &&
+ !profile.GetRawInfo(ADDRESS_HOME_STATE).empty() &&
+ !profile.GetRawInfo(ADDRESS_HOME_ZIP).empty();
}
// Return true if the |field_type| and |value| are valid within the context
@@ -596,19 +596,19 @@ bool PersonalDataManager::IsValidLearnableProfile(
if (!IsMinimumAddress(profile))
return false;
- string16 email = profile.GetInfo(EMAIL_ADDRESS);
+ string16 email = profile.GetRawInfo(EMAIL_ADDRESS);
if (!email.empty() && !IsValidEmail(email))
return false;
// Reject profiles with invalid US state information.
- string16 state = profile.GetInfo(ADDRESS_HOME_STATE);
+ string16 state = profile.GetRawInfo(ADDRESS_HOME_STATE);
if (profile.CountryCode() == "US" &&
!state.empty() && !autofill::IsValidState(state)) {
return false;
}
// Reject profiles with invalid US zip information.
- string16 zip = profile.GetInfo(ADDRESS_HOME_ZIP);
+ string16 zip = profile.GetRawInfo(ADDRESS_HOME_ZIP);
if (profile.CountryCode() == "US" && !zip.empty() && !IsValidZip(zip))
return false;
« no previous file with comments | « chrome/browser/autofill/form_group.cc ('k') | chrome/browser/autofill/personal_data_manager_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698