Skip to content

Payment Request: use InvalidState for non-fully active docs#58210

Merged
marcoscaceres merged 8 commits into
masterfrom
payments_invalid_state
Mar 4, 2026
Merged

Payment Request: use InvalidState for non-fully active docs#58210
marcoscaceres merged 8 commits into
masterfrom
payments_invalid_state

Conversation

@marcoscaceres
Copy link
Copy Markdown
Contributor

@marcoscaceres marcoscaceres commented Mar 3, 2026

Tests for w3c/payment-request#1056

  • also, AbortError, when frame is navigated or removed (making it non-fully active)

Copy link
Copy Markdown
Member

@romandev romandev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm with nit

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Should be (button, methodName, ...args)

Comment thread payment-request/META.yml
@marcoscaceres marcoscaceres enabled auto-merge (squash) March 4, 2026 03:41
@marcoscaceres marcoscaceres merged commit 13f625d into master Mar 4, 2026
21 of 22 checks passed
@marcoscaceres marcoscaceres deleted the payments_invalid_state branch March 4, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants