| Index: components/autofill/core/browser/contact_info.cc
|
| diff --git a/components/autofill/core/browser/contact_info.cc b/components/autofill/core/browser/contact_info.cc
|
| index 202867613b5f57ef4a6b4262af19b87e55da6e40..979434fc564a37c834371a8e0adc7fdf49280fdb 100644
|
| --- a/components/autofill/core/browser/contact_info.cc
|
| +++ b/components/autofill/core/browser/contact_info.cc
|
| @@ -52,38 +52,53 @@ void NameInfo::GetSupportedTypes(NativeFieldTypeSet* supported_types) const {
|
| }
|
|
|
| base::string16 NameInfo::GetRawInfo(NativeFieldType type) const {
|
| - type = AutofillType::GetEquivalentFieldType(type);
|
| - if (type == NAME_FIRST)
|
| - return first();
|
| + // TODO(isherman): Is GetEquivalentNativeType even necessary?
|
| + switch (AutofillType(type).GetEquivalentNativeType()) {
|
| + case NAME_FIRST:
|
| + return first();
|
|
|
| - if (type == NAME_MIDDLE)
|
| - return middle();
|
| + case NAME_MIDDLE:
|
| + return middle();
|
|
|
| - if (type == NAME_LAST)
|
| - return last();
|
| + case NAME_LAST:
|
| + return last();
|
|
|
| - if (type == NAME_MIDDLE_INITIAL)
|
| - return MiddleInitial();
|
| + case NAME_MIDDLE_INITIAL:
|
| + return MiddleInitial();
|
|
|
| - if (type == NAME_FULL)
|
| - return FullName();
|
| + case NAME_FULL:
|
| + return FullName();
|
|
|
| - return base::string16();
|
| + default:
|
| + return base::string16();
|
| + }
|
| }
|
|
|
| void NameInfo::SetRawInfo(NativeFieldType type, const base::string16& value) {
|
| - type = AutofillType::GetEquivalentFieldType(type);
|
| - DCHECK_EQ(NAME, AutofillType(type).group());
|
| - if (type == NAME_FIRST)
|
| - first_ = value;
|
| - else if (type == NAME_MIDDLE || type == NAME_MIDDLE_INITIAL)
|
| - middle_ = value;
|
| - else if (type == NAME_LAST)
|
| - last_ = value;
|
| - else if (type == NAME_FULL)
|
| - SetFullName(value);
|
| - else
|
| - NOTREACHED();
|
| + // TODO(isherman): Is GetEquivalentNativeType even necessary?
|
| + NativeFieldType native_type = AutofillType(type).GetEquivalentNativeType();
|
| + DCHECK_EQ(NAME, AutofillType(native_type).group());
|
| + switch (native_type) {
|
| + case NAME_FIRST:
|
| + first_ = value;
|
| + break;
|
| +
|
| + case NAME_MIDDLE:
|
| + case NAME_MIDDLE_INITIAL:
|
| + middle_ = value;
|
| + break;
|
| +
|
| + case NAME_LAST:
|
| + last_ = value;
|
| + break;
|
| +
|
| + case NAME_FULL:
|
| + SetFullName(value);
|
| + break;
|
| +
|
| + default:
|
| + NOTREACHED();
|
| + }
|
| }
|
|
|
| base::string16 NameInfo::FullName() const {
|
|
|