Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Spec] Only allow show() to be called in a foreground tab #1005

Merged
merged 4 commits into from
May 30, 2023

Conversation

stephenmcgruer
Copy link
Collaborator

@stephenmcgruer stephenmcgruer commented Apr 13, 2023

This addresses this issue in SPC - https://w3c.github.io/secure-payment-confirmation/#issue-e92619e9

The behavior spec'd in SPC appears to already be followed in general Payment Request implementations already, based on testing in both Safari and Chrome, so it seems safe to just spec it here instead.

The following tasks have been completed:

  • Modified Web platform tests (link)

Implementation commitment:

  • WebKit (already passes test)
  • Chromium (already passes test)
  • Gecko (doesn't implement Payment Request)

Preview | Diff

@stephenmcgruer

This comment was marked as outdated.

stephenmcgruer and others added 2 commits May 29, 2023 10:59
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
This reverts commit fe07e08.
@stephenmcgruer
Copy link
Collaborator Author

Build failure appears unrelated to this change, so I'm going to merge it.

@stephenmcgruer stephenmcgruer merged commit cce8f5e into gh-pages May 30, 2023
@stephenmcgruer stephenmcgruer deleted the smcgruer/reject-if-background branch May 30, 2023 13:22
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Jun 13, 2023
… called in a foreground tab, a=testonly

Automatic update from web-platform-tests
[PaymentRequest] Only allow show() to be called in a foreground tab (#40256)

See w3c/payment-request#1005

--

wpt-commits: ce9e78b850380c6bc782efeae6faaded29d0228e
wpt-pr: 40256
ErichDonGubler pushed a commit to erichdongubler-mozilla/firefox that referenced this pull request Jun 14, 2023
… called in a foreground tab, a=testonly

Automatic update from web-platform-tests
[PaymentRequest] Only allow show() to be called in a foreground tab (#40256)

See w3c/payment-request#1005

--

wpt-commits: ce9e78b850380c6bc782efeae6faaded29d0228e
wpt-pr: 40256
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Jun 16, 2023
… called in a foreground tab, a=testonly

Automatic update from web-platform-tests
[PaymentRequest] Only allow show() to be called in a foreground tab (#40256)

See w3c/payment-request#1005

--

wpt-commits: ce9e78b850380c6bc782efeae6faaded29d0228e
wpt-pr: 40256

UltraBlame original commit: 1200c0a52171decd28e41fe24ef83c66ad347680
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Jun 16, 2023
… called in a foreground tab, a=testonly

Automatic update from web-platform-tests
[PaymentRequest] Only allow show() to be called in a foreground tab (#40256)

See w3c/payment-request#1005

--

wpt-commits: ce9e78b850380c6bc782efeae6faaded29d0228e
wpt-pr: 40256

UltraBlame original commit: 1200c0a52171decd28e41fe24ef83c66ad347680
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Jun 16, 2023
… called in a foreground tab, a=testonly

Automatic update from web-platform-tests
[PaymentRequest] Only allow show() to be called in a foreground tab (#40256)

See w3c/payment-request#1005

--

wpt-commits: ce9e78b850380c6bc782efeae6faaded29d0228e
wpt-pr: 40256

UltraBlame original commit: 1200c0a52171decd28e41fe24ef83c66ad347680
marcoscaceres pushed a commit that referenced this pull request Feb 5, 2024
This addresses this issue in SPC - https://w3c.github.io/secure-payment-confirmation/#issue-e92619e9

The behavior spec'd in SPC appears to already be followed in general Payment Request implementations already, based on testing in both Safari and Chrome, so it seems safe to just spec it here instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants