| 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 a7357d69ead0c3f2f538d6d3781ad7871f83b2a3..970946f5367fd905f16a636f994bbb72c2e41e4d 100644
|
| --- a/components/autofill/core/browser/phone_number.cc
|
| +++ b/components/autofill/core/browser/phone_number.cc
|
| @@ -74,8 +74,8 @@ void PhoneNumber::GetSupportedTypes(NativeFieldTypeSet* supported_types) const {
|
| }
|
|
|
| base::string16 PhoneNumber::GetRawInfo(NativeFieldType type) const {
|
| - type = AutofillType::GetEquivalentFieldType(type);
|
| - if (type == PHONE_HOME_WHOLE_NUMBER)
|
| + // TODO(isherman): Is GetEquivalentNativeType even necessary?
|
| + if (AutofillType(type).GetEquivalentNativeType() == 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(NativeFieldType type) const {
|
|
|
| void PhoneNumber::SetRawInfo(NativeFieldType type,
|
| const base::string16& value) {
|
| - type = AutofillType::GetEquivalentFieldType(type);
|
| - if (type != PHONE_HOME_CITY_AND_NUMBER &&
|
| - type != PHONE_HOME_WHOLE_NUMBER) {
|
| + // TODO(isherman): Is GetEquivalentNativeType even necessary?
|
| + type = AutofillType(type).GetEquivalentNativeType();
|
| + 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,8 +106,7 @@ void PhoneNumber::SetRawInfo(NativeFieldType 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 {
|
| - NativeFieldType native_type =
|
| - AutofillType::GetEquivalentFieldType(type.native_type());
|
| + NativeFieldType native_type = type.GetEquivalentNativeType();
|
| UpdateCacheIfNeeded(app_locale);
|
|
|
| // Queries for whole numbers will return the non-normalized number if
|
| @@ -143,8 +142,7 @@ base::string16 PhoneNumber::GetInfo(const AutofillType& type,
|
| bool PhoneNumber::SetInfo(const AutofillType& type,
|
| const base::string16& value,
|
| const std::string& app_locale) {
|
| - NativeFieldType native_type =
|
| - AutofillType::GetEquivalentFieldType(type.native_type());
|
| + NativeFieldType native_type = type.GetEquivalentNativeType();
|
| SetRawInfo(native_type, value);
|
|
|
| if (number_.empty())
|
| @@ -196,10 +194,9 @@ 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) {
|
| - NativeFieldType native_field_type =
|
| - AutofillType::GetEquivalentFieldType(field_type.native_type());
|
| + NativeFieldType native_field_type = type.GetEquivalentNativeType();
|
| if (native_field_type == PHONE_HOME_COUNTRY_CODE) {
|
| country_ = value;
|
| return true;
|
|
|