-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Check for transferable detachedness after serializing #3557
Conversation
Since serializing can make a transferable detached, it seems better to throw for that later. This matches Chrome/Safari. Firefox ends up with a detached transferable. Edge crashes. See https://bugs.chromium.org/p/chromium/issues/detail?id=816447 for more context. Tests: ... (should update web-platform-tests/wpt#9672)
Before updating the tests I'd like agreement on this change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM editorially; nice find from Jeremy. I guess you'll still want to update the acks.
Thanks, also need to add Jeremy's test (and flip the conditions of some of the other ones in that PR). |
FYI: browser bugs are linked from the test PR. |
…tonly Automatic update from web-platform-testsTest postMessage() exception order For whatwg/html#3508 and whatwg/html#3557. wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672 wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672
…tonly Automatic update from web-platform-testsTest postMessage() exception order For whatwg/html#3508 and whatwg/html#3557. wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672 wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672 UltraBlame original commit: 7aa5cbfd0daa25162b40ca204419fa9fe82049a6
…tonly Automatic update from web-platform-testsTest postMessage() exception order For whatwg/html#3508 and whatwg/html#3557. wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672 wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672 UltraBlame original commit: 7aa5cbfd0daa25162b40ca204419fa9fe82049a6
…tonly Automatic update from web-platform-testsTest postMessage() exception order For whatwg/html#3508 and whatwg/html#3557. wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672 wpt-commits: fc04bb5708878ee7b68996f8a161507022e07777 wpt-pr: 9672 UltraBlame original commit: 7aa5cbfd0daa25162b40ca204419fa9fe82049a6
Since serializing can make a transferable detached, it seems better to throw for that later.
This matches Chrome/Safari. Firefox ends up with a detached transferable. Edge crashes.
See https://bugs.chromium.org/p/chromium/issues/detail?id=816447 for more context.
Tests: ... (should update web-platform-tests/wpt#9672)
/acknowledgements.html ( diff )
/structured-data.html ( diff )