Update transient activation to match WebKit

w3c / payment-request

Payment Request API

https://www.w3.org/TR/payment-request-1.1/

Other

482 stars 183 forks source link

Update transient activation to match WebKit #961

Closed marcoscaceres closed 2 years ago

marcoscaceres commented 2 years ago

Update the spec to match what WebKit does. We can then update Chrome to match.

The following tasks have been completed:

[x] Modified Web platform tests

Implementation commitment:

[X] Safari - already implemented
- [ ] Chrome

Optional, impact on Payment Handler spec?

none

Preview | Diff

marcoscaceres commented 2 years ago

So... SecurityError LGTM? Maybe something the web should standardize on ;)

I've asked for this too elsewhere, but wasn't successful... as you point out, it's somewhat specific to the API (and if the API throws/rejects or can get away with doing nothing).

marcoscaceres commented 2 years ago

Could someone please also approve the tests? 🙏 https://github.com/web-platform-tests/wpt/pull/30006

marcoscaceres commented 2 years ago

@stephenmcgruer, filed Chrome bug https://bugs.chromium.org/p/chromium/issues/detail?id=1239453

Happy to send a patch, but please see my email as I'm not sure how to enable the right flag to trigger that code path.