Don't show error on unknown-result payment transactions |
|||||||
Issue description
Only PaymentResponse.complete("fail") should show an error dialog.
Calling PaymentResponse.complete("success"), PaymentResponse.complete("unknown"), and PaymentResponse.complete() should not show an error dialog.
,
Jun 16 2017
Hello, I would like to merge this into M-60 to enable shipping PaymentRequest on desktop with correct implementation of the spec.
,
Jun 17 2017
This bug requires manual review: M60 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 20 2017
rouslan@ - Could you please provide a sample test file or sample URL to test the issue. Thanks...!!
,
Jun 20 2017
Koiredla, the test URL https://rsolomakhin.github.io/pr/type/credit/. Type in the correct CVC number for your credit card: it should show the credit card number on the page in green text after my patch. Beware that the card number is not masked on that webpage, so beware shoulder surfers. That is a demo website and no payment is made. Before my patch, the page always showed "Transaction failed" error message in the payment dialog, even if you typed in the correct CVC number.
,
Jun 20 2017
,
Jun 21 2017
,
Jun 27 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by bugdroid1@chromium.org
, Jun 16 2017