New issue
Advanced search Search tips

Issue 608222 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug

Blocking:
issue 587995



Sign in to add a comment

Provide shipping in PaymentRequest result

Project Member Reported by rouslan@chromium.org, May 1 2016

Issue description

If the merchant provides non-NULL shipping options, the merchant does    
not need shipping address to calculate shipping costs. The UI selects    
the default shipping address, but does not send it to the merchant. PaymentRequest should provide shipping details to the merchant when the user clicks "Pay". This does not happen.

This patch is for Android, the first platform with PaymentRequest UI.
 
Labels: -Pri-3 Pri-1
Project Member

Comment 2 by bugdroid1@chromium.org, May 3 2016

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

commit f63c3bce0928c9534671fb325d45fd1524ac6b37
Author: rouslan <rouslan@chromium.org>
Date: Tue May 03 01:02:50 2016

Provide shipping in PaymentRequest result

If the merchant provides non-NULL shipping options, the merchant does
not need shipping address to calculate shipping costs. The UI selects
the default shipping address, but does not send it to the merchant. This
patch sends the shipping address to the merchant when the user clicks
"Pay".

BUG= 608222 

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

[modify] https://crrev.com/f63c3bce0928c9534671fb325d45fd1524ac6b37/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[modify] https://crrev.com/f63c3bce0928c9534671fb325d45fd1524ac6b37/third_party/WebKit/Source/modules/payments/PaymentRequest.cpp
[modify] https://crrev.com/f63c3bce0928c9534671fb325d45fd1524ac6b37/third_party/WebKit/Source/modules/payments/PaymentsValidators.cpp
[modify] https://crrev.com/f63c3bce0928c9534671fb325d45fd1524ac6b37/third_party/WebKit/Source/modules/payments/PaymentsValidators.h
[modify] https://crrev.com/f63c3bce0928c9534671fb325d45fd1524ac6b37/third_party/WebKit/Source/modules/payments/PaymentsValidatorsTest.cpp
[modify] https://crrev.com/f63c3bce0928c9534671fb325d45fd1524ac6b37/third_party/WebKit/public/platform/modules/payments/payment_request.mojom

Status: Fixed (was: Started)
Labels: M-53

Sign in to add a comment