Commit 4cf2f73e authored by Rouslan Solomakhin's avatar Rouslan Solomakhin Committed by Commit Bot

[Payment Request] Don't crash on multiple CanMakePayment responses.

Bug: 891371
Change-Id: Iea90f598eac5693b6797efa9ca91cabf7b097326
Reviewed-on: https://chromium-review.googlesource.com/c/1258023
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: default avatarGanggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596020}
parent 699ac366
...@@ -1341,7 +1341,10 @@ void PaymentRequest::OnAbort(bool aborted_successfully) { ...@@ -1341,7 +1341,10 @@ void PaymentRequest::OnAbort(bool aborted_successfully) {
} }
void PaymentRequest::OnCanMakePayment(CanMakePaymentQueryResult result) { void PaymentRequest::OnCanMakePayment(CanMakePaymentQueryResult result) {
DCHECK(can_make_payment_resolver_); // TODO(https://crbug.com/891371): Understand how the resolver could be null
// here and prevent it.
if (!can_make_payment_resolver_)
return;
switch (result) { switch (result) {
case CanMakePaymentQueryResult::WARNING_CAN_MAKE_PAYMENT: case CanMakePaymentQueryResult::WARNING_CAN_MAKE_PAYMENT:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment