New issue
Advanced search Search tips

Issue 715597 link

Starred by 1 user
Status: Fixed
Owner:
se...@chromium.org
Closed: Apr 2017
Cc:
anthonyvd@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Make sure the user cannot pay while shipping address is validated by the merchant.

Project Member Reported by se...@chromium.org, Apr 26 2017 
When a new shipping address is selected, there is a time where we normalize and send to the merchant for validation.

From the time the user selects an address and the time where we get the callback from the merchant, the user should not be allowed to click pay.
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 27 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb

commit 7e61913aaeda21fcfdbefb435d6085efdc1aa9fb
Author: sebsg <sebsg@chromium.org>
Date: Thu Apr 27 16:07:10 2017

[Payments] Normalize shipping address change on Desktop

Also make sure that the user cannot click pay until we
get the callback from the merchant.

BUG= 715170 , 715597 

Review-Url: https://codereview.chromium.org/2836353002
Cr-Commit-Position: refs/heads/master@{#467698}

[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/content/BUILD.gn
[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/content/payment_request_state.cc
[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/content/payment_request_state.h
[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/content/payment_request_state_unittest.cc
[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/content/payment_response_helper.cc
[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/content/payment_response_helper.h
[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/content/payment_response_helper_unittest.cc
[modify] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/core/BUILD.gn
[add] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/core/test_address_normalizer.cc
[add] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/core/test_address_normalizer.h
[add] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/core/test_payment_request_delegate.cc
[add] https://crrev.com/7e61913aaeda21fcfdbefb435d6085efdc1aa9fb/components/payments/core/test_payment_request_delegate.h

Comment 2 by se...@chromium.org, Apr 27 2017

Status: Fixed (was: Started)

Comment 3 by lafo...@chromium.org, Jun 27 2017

Components: -UI>Browser>Autofill>Payments UI>Browser>Payments
►

Sign in to add a comment