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

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

Issue 298863012: Use upstream libaddressinput in Chrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Self review. Created 6 years, 6 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/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());

Powered by Google App Engine
This is Rietveld 408576698