| Index: components/autofill/core/browser/phone_number.cc
|
| diff --git a/components/autofill/core/browser/phone_number.cc b/components/autofill/core/browser/phone_number.cc
|
| index e452f9823cc41a9cc2637de277300f2bab488350..2ee67f98b72e732957b41dab4ff7110ca0f5d850 100644
|
| --- a/components/autofill/core/browser/phone_number.cc
|
| +++ b/components/autofill/core/browser/phone_number.cc
|
| @@ -74,8 +74,8 @@ void PhoneNumber::GetSupportedTypes(ServerFieldTypeSet* supported_types) const {
|
| }
|
|
|
| base::string16 PhoneNumber::GetRawInfo(ServerFieldType type) const {
|
| - type = AutofillType::GetEquivalentFieldType(type);
|
| - if (type == PHONE_HOME_WHOLE_NUMBER)
|
| + // TODO(isherman): Is GetStorableType even necessary?
|
| + if (AutofillType(type).GetStorableType() == PHONE_HOME_WHOLE_NUMBER)
|
| return number_;
|
|
|
| // Only the whole number is available as raw data. All of the other types are
|
| @@ -86,9 +86,9 @@ base::string16 PhoneNumber::GetRawInfo(ServerFieldType type) const {
|
|
|
| void PhoneNumber::SetRawInfo(ServerFieldType type,
|
| const base::string16& value) {
|
| - type = AutofillType::GetEquivalentFieldType(type);
|
| - if (type != PHONE_HOME_CITY_AND_NUMBER &&
|
| - type != PHONE_HOME_WHOLE_NUMBER) {
|
| + // TODO(isherman): Is GetStorableType even necessary?
|
| + type = AutofillType(type).GetStorableType();
|
| + if (type != PHONE_HOME_CITY_AND_NUMBER && type != PHONE_HOME_WHOLE_NUMBER) {
|
| // Only full phone numbers should be set directly. The remaining field
|
| // field types are read-only.
|
| return;
|
| @@ -106,18 +106,17 @@ void PhoneNumber::SetRawInfo(ServerFieldType type,
|
| // If the phone cannot be normalized, returns the stored value verbatim.
|
| base::string16 PhoneNumber::GetInfo(const AutofillType& type,
|
| const std::string& app_locale) const {
|
| - ServerFieldType server_type =
|
| - AutofillType::GetEquivalentFieldType(type.server_type());
|
| + ServerFieldType storable_type = type.GetStorableType();
|
| UpdateCacheIfNeeded(app_locale);
|
|
|
| // Queries for whole numbers will return the non-normalized number if
|
| // normalization for the number fails. All other field types require
|
| // normalization.
|
| - if (server_type != PHONE_HOME_WHOLE_NUMBER &&
|
| + if (storable_type != PHONE_HOME_WHOLE_NUMBER &&
|
| !cached_parsed_phone_.IsValidNumber())
|
| return base::string16();
|
|
|
| - switch (server_type) {
|
| + switch (storable_type) {
|
| case PHONE_HOME_WHOLE_NUMBER:
|
| return cached_parsed_phone_.GetWholeNumber();
|
|
|
| @@ -143,9 +142,7 @@ base::string16 PhoneNumber::GetInfo(const AutofillType& type,
|
| bool PhoneNumber::SetInfo(const AutofillType& type,
|
| const base::string16& value,
|
| const std::string& app_locale) {
|
| - ServerFieldType server_type =
|
| - AutofillType::GetEquivalentFieldType(type.server_type());
|
| - SetRawInfo(server_type, value);
|
| + SetRawInfo(type.GetStorableType(), value);
|
|
|
| if (number_.empty())
|
| return true;
|
| @@ -196,31 +193,30 @@ PhoneNumber::PhoneCombineHelper::PhoneCombineHelper() {
|
| PhoneNumber::PhoneCombineHelper::~PhoneCombineHelper() {
|
| }
|
|
|
| -bool PhoneNumber::PhoneCombineHelper::SetInfo(const AutofillType& field_type,
|
| +bool PhoneNumber::PhoneCombineHelper::SetInfo(const AutofillType& type,
|
| const base::string16& value) {
|
| - ServerFieldType server_field_type =
|
| - AutofillType::GetEquivalentFieldType(field_type.server_type());
|
| - if (server_field_type == PHONE_HOME_COUNTRY_CODE) {
|
| + ServerFieldType storable_type = type.GetStorableType();
|
| + if (storable_type == PHONE_HOME_COUNTRY_CODE) {
|
| country_ = value;
|
| return true;
|
| }
|
|
|
| - if (server_field_type == PHONE_HOME_CITY_CODE) {
|
| + if (storable_type == PHONE_HOME_CITY_CODE) {
|
| city_ = value;
|
| return true;
|
| }
|
|
|
| - if (server_field_type == PHONE_HOME_CITY_AND_NUMBER) {
|
| + if (storable_type == PHONE_HOME_CITY_AND_NUMBER) {
|
| phone_ = value;
|
| return true;
|
| }
|
|
|
| - if (server_field_type == PHONE_HOME_WHOLE_NUMBER) {
|
| + if (storable_type == PHONE_HOME_WHOLE_NUMBER) {
|
| whole_number_ = value;
|
| return true;
|
| }
|
|
|
| - if (server_field_type == PHONE_HOME_NUMBER) {
|
| + if (storable_type == PHONE_HOME_NUMBER) {
|
| phone_.append(value);
|
| return true;
|
| }
|
|
|