Commit 44a0099e authored by Sahel Sharify's avatar Sahel Sharify Committed by Commit Bot

[Payments] secure-payment-confirmation(SPC) does not skip UI.

skipped_payment_request_ui_ should be false for SPC since Chrome's UI
is shown for the payment request.

Bug: 1117042
Change-Id: I1c7c4ff612e112d792ffc50ef9b161405c46ac12
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365979Reviewed-by: default avatarNick Burris <nburris@chromium.org>
Commit-Queue: Sahel Sharify <sahel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#800214}
parent 7b78e293
...@@ -628,8 +628,9 @@ bool PaymentRequest::OnlySingleAppCanProvideAllRequiredInformation() const { ...@@ -628,8 +628,9 @@ bool PaymentRequest::OnlySingleAppCanProvideAllRequiredInformation() const {
} }
bool PaymentRequest::SatisfiesSkipUIConstraints() { bool PaymentRequest::SatisfiesSkipUIConstraints() {
// Only allowing URL base payment apps to skip the payment sheet. // Only allowing URL based payment apps to skip the payment sheet.
skipped_payment_request_ui_ = skipped_payment_request_ui_ =
!spec()->IsSecurePaymentConfirmationRequested() &&
(spec()->url_payment_method_identifiers().size() > 0 || (spec()->url_payment_method_identifiers().size() > 0 ||
delegate_->SkipUiForBasicCard()) && delegate_->SkipUiForBasicCard()) &&
base::FeatureList::IsEnabled(features::kWebPaymentsSingleAppUiSkip) && base::FeatureList::IsEnabled(features::kWebPaymentsSingleAppUiSkip) &&
......
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