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

Unified Diff: chrome/browser/ui/autofill/data_model_wrapper_unittest.cc

Issue 25092011: rAc: update android test (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: git try Created 7 years, 3 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: chrome/browser/ui/autofill/data_model_wrapper_unittest.cc
diff --git a/chrome/browser/ui/autofill/data_model_wrapper_unittest.cc b/chrome/browser/ui/autofill/data_model_wrapper_unittest.cc
index f8b750f27795336843860101b4946ca6d21f7b02..68cfcc33291d0c1f21da5b33da1cceea01458781 100644
--- a/chrome/browser/ui/autofill/data_model_wrapper_unittest.cc
+++ b/chrome/browser/ui/autofill/data_model_wrapper_unittest.cc
@@ -103,4 +103,70 @@ TEST(DataModelWrapperTest, GetDisplayTextEmptyWithoutPhone) {
EXPECT_FALSE(address_wrapper.GetDisplayText(&unused, &unused2));
}
+TEST(DataModelWrapperTest, GetDisplayPhoneNumber) {
+ const base::string16 national_unformatted = ASCIIToUTF16("3104567890");
+ const base::string16 national_formatted = ASCIIToUTF16("(310) 456-7890");
+ const base::string16 international_unformatted = ASCIIToUTF16("13104567890");
+ const base::string16 international_formatted =
+ ASCIIToUTF16("+1 310-456-7890");
+ const base::string16 user_formatted = ASCIIToUTF16("310.456 78 90");
+
+ scoped_ptr<wallet::WalletItems::MaskedInstrument> instrument(
+ wallet::GetTestMaskedInstrument());
+ AutofillProfile profile(test::GetVerifiedProfile());
+
+ // No matter what format a wallet number is in, it gets formatted in a
+ // standard way.
+ WalletInstrumentWrapper instrument_wrapper(instrument.get());
+ const_cast<wallet::Address*>(&instrument->address())->
+ set_phone_number(national_unformatted);
+ EXPECT_EQ(national_formatted,
+ instrument_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+ WalletAddressWrapper address_wrapper(&instrument->address());
+ EXPECT_EQ(national_formatted,
+ address_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+ AutofillProfileWrapper profile_wrapper(&profile);
+
+ const_cast<wallet::Address*>(&instrument->address())->
+ set_phone_number(national_formatted);
+ EXPECT_EQ(national_formatted,
+ instrument_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+ EXPECT_EQ(national_formatted,
+ address_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+
+ const_cast<wallet::Address*>(&instrument->address())->
+ set_phone_number(international_unformatted);
+ EXPECT_EQ(national_formatted,
+ instrument_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+ EXPECT_EQ(national_formatted,
+ address_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+
+ // Autofill numbers that are unformatted get formatted either nationally or
+ // internationally depending on the presence of a country code. Formatted
+ // numbers stay formatted.
+ profile.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, international_unformatted);
+ EXPECT_EQ(international_formatted,
+ profile_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+ profile.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, national_unformatted);
+ EXPECT_EQ(national_formatted,
+ profile_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+ profile.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, national_formatted);
+ EXPECT_EQ(national_formatted,
+ profile_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+ profile.SetRawInfo(PHONE_HOME_WHOLE_NUMBER, user_formatted);
+ EXPECT_EQ(user_formatted,
+ profile_wrapper.GetInfoForDisplay(
+ AutofillType(PHONE_HOME_WHOLE_NUMBER)));
+
+}
+
} // namespace autofill

Powered by Google App Engine
This is Rietveld 408576698