Chromium Code Reviews| Index: chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc b/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| index 769ffaa763d6b2adc00a0c7c929d2be897f53587..901d385cef1d5114e39633544d373181097a6855 100644 |
| --- a/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| +++ b/chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc |
| @@ -3,7 +3,9 @@ |
| // found in the LICENSE file. |
| #include <map> |
| +#include <utility> |
| +#include "base/basictypes.h" |
| #include "base/bind.h" |
| #include "base/bind_helpers.h" |
| #include "base/callback.h" |
| @@ -54,8 +56,10 @@ |
| #include "grit/generated_resources.h" |
| #include "testing/gmock/include/gmock/gmock.h" |
| #include "testing/gtest/include/gtest/gtest.h" |
| -#include "third_party/libaddressinput/chromium/cpp/include/libaddressinput/address_data.h" |
| -#include "third_party/libaddressinput/chromium/cpp/include/libaddressinput/address_validator.h" |
| +#include "third_party/libaddressinput/chromium/preload_address_validator.h" |
| +#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_field.h" |
| +#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_problem.h" |
| +#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_validator.h" |
| #include "ui/base/resource/resource_bundle.h" |
| #if defined(OS_WIN) |
| @@ -69,11 +73,7 @@ namespace autofill { |
| namespace { |
| -using ::i18n::addressinput::AddressData; |
| -using ::i18n::addressinput::AddressProblemFilter; |
| -using ::i18n::addressinput::AddressProblem; |
| -using ::i18n::addressinput::AddressProblems; |
| -using ::i18n::addressinput::AddressValidator; |
| +using ::i18n::addressinput::FieldProblemMap; |
| using testing::AtLeast; |
| using testing::DoAll; |
| using testing::Return; |
| @@ -343,7 +343,7 @@ class TestAutofillDialogController |
| GetTestingManager(); |
| } |
| - virtual AddressValidator* GetValidator() OVERRIDE { |
| + virtual PreloadAddressValidator* GetValidator() OVERRIDE { |
| return &mock_validator_; |
| } |
| @@ -1008,15 +1008,13 @@ TEST_F(AutofillDialogControllerTest, AutofillProfilesPopInvalidIntoEdit) { |
| // Now make up a problem and make sure the profile isn't in the list. |
| Reset(); |
| SwitchToAutofill(); |
| - AddressProblems problems; |
| - problems.push_back( |
| - AddressProblem(::i18n::addressinput::POSTAL_CODE, |
| - AddressProblem::MISMATCHING_VALUE, |
| - IDS_LEARN_MORE)); |
| + FieldProblemMap problems; |
| + problems.insert(std::make_pair(::i18n::addressinput::POSTAL_CODE, |
| + ::i18n::addressinput::MISMATCHING_VALUE)); |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - ValidateAddress(CountryCodeMatcher("US"), _, _)). |
| + Validate(CountryCodeMatcher("US"), _, _)). |
| WillRepeatedly(DoAll(SetArgPointee<2>(problems), |
| - Return(AddressValidator::SUCCESS))); |
| + Return(PreloadAddressValidator::SUCCESS))); |
| controller()->GetTestingManager()->AddTestingProfile(&verified_profile); |
| shipping_model = GetMenuModelForSection(SECTION_SHIPPING); |
| @@ -1040,17 +1038,15 @@ TEST_F(AutofillDialogControllerTest, AutofillProfilesRevalidateAfterRulesLoad) { |
| EXPECT_FALSE(controller()->IsManuallyEditingSection(SECTION_SHIPPING)); |
| EXPECT_FALSE(controller()->IsManuallyEditingSection(SECTION_BILLING)); |
| - AddressProblems problems; |
| - problems.push_back( |
| - AddressProblem(::i18n::addressinput::POSTAL_CODE, |
| - AddressProblem::MISMATCHING_VALUE, |
| - IDS_LEARN_MORE)); |
| + FieldProblemMap problems; |
| + problems.insert(std::make_pair(::i18n::addressinput::POSTAL_CODE, |
| + ::i18n::addressinput::MISMATCHING_VALUE)); |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - ValidateAddress(CountryCodeMatcher("US"), _, _)). |
| + Validate(CountryCodeMatcher("US"), _, _)). |
| WillRepeatedly(DoAll(SetArgPointee<2>(problems), |
| - Return(AddressValidator::SUCCESS))); |
| + Return(PreloadAddressValidator::SUCCESS))); |
| - controller()->OnAddressValidationRulesLoaded("US", true); |
| + controller()->AddressValidationRulesLoaded(true, "US", 0); |
| EXPECT_EQ(4, shipping_model->GetItemCount()); |
| EXPECT_TRUE(controller()->IsManuallyEditingSection(SECTION_SHIPPING)); |
| EXPECT_TRUE(controller()->IsManuallyEditingSection(SECTION_BILLING)); |
| @@ -3208,14 +3204,14 @@ TEST_F(AutofillDialogControllerTest, CountryChangeUpdatesSection) { |
| TEST_F(AutofillDialogControllerTest, CorrectCountryFromInputs) { |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - ValidateAddress(CountryCodeMatcher("DE"), _, _)); |
| + Validate(CountryCodeMatcher("DE"), _, _)); |
| FieldValueMap billing_inputs; |
| billing_inputs[ADDRESS_BILLING_COUNTRY] = ASCIIToUTF16("Germany"); |
| controller()->InputsAreValid(SECTION_BILLING, billing_inputs); |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - ValidateAddress(CountryCodeMatcher("FR"), _, _)); |
| + Validate(CountryCodeMatcher("FR"), _, _)); |
| FieldValueMap shipping_inputs; |
| shipping_inputs[ADDRESS_HOME_COUNTRY] = ASCIIToUTF16("France"); |
| @@ -3225,10 +3221,10 @@ TEST_F(AutofillDialogControllerTest, CorrectCountryFromInputs) { |
| TEST_F(AutofillDialogControllerTest, ValidationRulesLoadedOnCountryChange) { |
| ResetControllerWithFormData(DefaultFormData()); |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - LoadRules("US")).Times(AtLeast(1)); |
| + LoadRules("US", _)).Times(AtLeast(1)); |
|
please use gerrit instead
2014/06/05 22:22:47
To reduce reviewer burden: Make LoadRules() not ta
please use gerrit instead
2014/06/09 23:28:16
Done.
|
| controller()->Show(); |
| - EXPECT_CALL(*controller()->GetMockValidator(), LoadRules("FR")); |
| + EXPECT_CALL(*controller()->GetMockValidator(), LoadRules("FR", _)); |
| controller()->UserEditedOrActivatedInput(SECTION_BILLING, |
| ADDRESS_BILLING_COUNTRY, |
| gfx::NativeView(), |
| @@ -3242,9 +3238,9 @@ TEST_F(AutofillDialogControllerTest, UsValidationRulesLoadedForJpOnlyProfile) { |
| AutofillProfile jp_profile(base::GenerateGUID(), kSettingsOrigin); |
| jp_profile.SetRawInfo(ADDRESS_HOME_COUNTRY, ASCIIToUTF16("JP")); |
| controller()->GetTestingManager()->AddTestingProfile(&jp_profile); |
| - EXPECT_CALL(*controller()->GetMockValidator(), LoadRules("US")); |
| + EXPECT_CALL(*controller()->GetMockValidator(), LoadRules("US", _)); |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - LoadRules("JP")).Times(AtLeast(1)); |
| + LoadRules("JP", _)).Times(AtLeast(1)); |
| controller()->Show(); |
| } |
| @@ -3254,8 +3250,8 @@ TEST_F(AutofillDialogControllerTest, InvalidWhenRulesNotReady) { |
| // If the rules haven't loaded yet, validation errors should show on submit. |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - ValidateAddress(CountryCodeMatcher("US"), _, _)). |
| - WillRepeatedly(Return(AddressValidator::RULES_NOT_READY)); |
| + Validate(CountryCodeMatcher("US"), _, _)). |
| + WillRepeatedly(Return(PreloadAddressValidator::RULES_NOT_READY)); |
| FieldValueMap inputs; |
| inputs[ADDRESS_HOME_ZIP] = ASCIIToUTF16("1234"); |
| @@ -3284,8 +3280,8 @@ TEST_F(AutofillDialogControllerTest, ValidButUnverifiedWhenRulesFail) { |
| // If the rules are unavailable, validation errors should not show. |
| EXPECT_CALL(*controller()->GetMockValidator(), |
| - ValidateAddress(CountryCodeMatcher("US"), _, _)). |
| - WillRepeatedly(Return(AddressValidator::RULES_UNAVAILABLE)); |
| + Validate(CountryCodeMatcher("US"), _, _)) |
| + .WillRepeatedly(Return(PreloadAddressValidator::RULES_UNAVAILABLE)); |
| FieldValueMap outputs; |
| AutofillProfile full_profile(test::GetFullProfile()); |