| Index: components/payments/content/payment_response_helper_unittest.cc
|
| diff --git a/components/payments/content/payment_response_helper_unittest.cc b/components/payments/content/payment_response_helper_unittest.cc
|
| index 7c94919ec37ab78cc34f6e3b670c337fdb7f6786..ed9757c551825cfb04a5c2f4d5eca236824d3811 100644
|
| --- a/components/payments/content/payment_response_helper_unittest.cc
|
| +++ b/components/payments/content/payment_response_helper_unittest.cc
|
| @@ -23,25 +23,6 @@
|
|
|
| namespace payments {
|
|
|
| -class FakeAddressNormalizer : public AddressNormalizer {
|
| - public:
|
| - FakeAddressNormalizer() : AddressNormalizer(nullptr, nullptr) {}
|
| -
|
| - void LoadRulesForRegion(const std::string& region_code) override {}
|
| -
|
| - bool AreRulesLoadedForRegion(const std::string& region_code) override {
|
| - return true;
|
| - }
|
| -
|
| - void StartAddressNormalization(
|
| - const autofill::AutofillProfile& profile,
|
| - const std::string& region_code,
|
| - int timeout_seconds,
|
| - AddressNormalizer::Delegate* requester) override {
|
| - requester->OnAddressNormalized(profile);
|
| - }
|
| -};
|
| -
|
| class FakePaymentRequestDelegate : public PaymentRequestDelegate {
|
| public:
|
| FakePaymentRequestDelegate(
|
| @@ -77,7 +58,7 @@
|
| base::ASCIIToUTF16("123"));
|
| }
|
|
|
| - std::unique_ptr<::i18n::addressinput::Source> GetAddressInputSource()
|
| + std::unique_ptr<const ::i18n::addressinput::Source> GetAddressInputSource()
|
| override {
|
| return nullptr;
|
| }
|
| @@ -85,17 +66,12 @@
|
| std::unique_ptr<::i18n::addressinput::Storage> GetAddressInputStorage()
|
| override {
|
| return nullptr;
|
| - }
|
| -
|
| - AddressNormalizer* GetAddressNormalizer() override {
|
| - return &address_normalizer_;
|
| }
|
|
|
| private:
|
| autofill::PersonalDataManager* personal_data_manager_;
|
| std::string locale_;
|
| const GURL last_committed_url_;
|
| - FakeAddressNormalizer address_normalizer_;
|
| DISALLOW_COPY_AND_ASSIGN(FakePaymentRequestDelegate);
|
| };
|
|
|
| @@ -164,9 +140,6 @@
|
| const mojom::PaymentResponsePtr& response() { return payment_response_; }
|
| autofill::AutofillProfile* test_address() { return &address_; }
|
| PaymentInstrument* test_instrument() { return autofill_instrument_.get(); }
|
| - PaymentRequestDelegate* test_payment_request_delegate() {
|
| - return &payment_request_delegate_;
|
| - }
|
|
|
| private:
|
| std::unique_ptr<PaymentRequestSpec> spec_;
|
| @@ -189,8 +162,7 @@
|
| // "visa" is specified directly in the supportedMethods so it is returned
|
| // as the method name.
|
| PaymentResponseHelper helper("en-US", spec(), test_instrument(),
|
| - test_payment_request_delegate(), test_address(),
|
| - test_address(), this);
|
| + test_address(), test_address(), this);
|
| EXPECT_EQ("visa", response()->method_name);
|
| EXPECT_EQ(
|
| "{\"billingAddress\":"
|
| @@ -226,8 +198,7 @@
|
| // TODO(mathp): Currently synchronous, when async will need a RunLoop.
|
| // "basic-card" is specified so it is returned as the method name.
|
| PaymentResponseHelper helper("en-US", spec(), test_instrument(),
|
| - test_payment_request_delegate(), test_address(),
|
| - test_address(), this);
|
| + test_address(), test_address(), this);
|
| EXPECT_EQ("basic-card", response()->method_name);
|
| EXPECT_EQ(
|
| "{\"billingAddress\":"
|
| @@ -264,8 +235,7 @@
|
| GetMethodDataForVisa());
|
|
|
| PaymentResponseHelper helper("en-US", spec(), test_instrument(),
|
| - test_payment_request_delegate(), test_address(),
|
| - test_address(), this);
|
| + test_address(), test_address(), this);
|
|
|
| // Check that all the expected values were set.
|
| EXPECT_EQ("US", response()->shipping_address->country);
|
| @@ -293,8 +263,7 @@
|
| RecreateSpecWithOptions(std::move(options));
|
|
|
| PaymentResponseHelper helper("en-US", spec(), test_instrument(),
|
| - test_payment_request_delegate(), test_address(),
|
| - test_address(), this);
|
| + test_address(), test_address(), this);
|
|
|
| // Check that all the expected values were set.
|
| EXPECT_EQ("John H. Doe", response()->payer_name.value());
|
| @@ -311,8 +280,7 @@
|
| RecreateSpecWithOptions(std::move(options));
|
|
|
| PaymentResponseHelper helper("en-US", spec(), test_instrument(),
|
| - test_payment_request_delegate(), test_address(),
|
| - test_address(), this);
|
| + test_address(), test_address(), this);
|
|
|
| // Check that the name was set, but not the other values.
|
| EXPECT_EQ("John H. Doe", response()->payer_name.value());
|
| @@ -332,8 +300,7 @@
|
| RecreateSpecWithOptions(std::move(options));
|
|
|
| PaymentResponseHelper helper("en-US", spec(), test_instrument(),
|
| - test_payment_request_delegate(), test_address(),
|
| - test_address(), this);
|
| + test_address(), test_address(), this);
|
|
|
| // Check that the phone was formatted.
|
| EXPECT_EQ("+15151231234", response()->payer_phone.value());
|
|
|