Implement self.structuredClone()
Categories
(Core :: DOM: postMessage, enhancement)
Tracking
()
Tracking | Status | |
---|---|---|
firefox94 | --- | fixed |
People
(Reporter: d, Assigned: evilpie)
References
(Blocks 2 open bugs, )
Details
(Keywords: dev-doc-complete)
Attachments
(3 files)
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
Updated•2 years ago
|
Assignee | ||
Comment 2•2 years ago
|
||
Steve, I was hoping you could take a look at this. Thanks.
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Comment 3•2 years ago
|
||
Pushed by evilpies@gmail.com: https://hg.mozilla.org/integration/autoland/rev/4d0e3d6a1960 Implement self.structuredClone(). r=smaug,sfink,baku https://hg.mozilla.org/integration/autoland/rev/d31a1c199922 Rename PostMessageOptions to StructuredSerializeOptions. r=smaug
Comment 5•2 years ago
|
||
Backed out for causing wpt failures in structured-clone.any.worker.html
Assignee | ||
Comment 6•2 years ago
•
|
||
The destructor has an assertion for mTransferredPorts.IsEmpty()
. As far as I can tell we aren't supposed to do anything with the transferred ports, so we can just work around that assertion.
Pushed by evilpies@gmail.com: https://hg.mozilla.org/integration/autoland/rev/f50502328881 Implement self.structuredClone(). r=smaug,sfink,baku https://hg.mozilla.org/integration/autoland/rev/b486e4108902 Rename PostMessageOptions to StructuredSerializeOptions. r=smaug
Assignee | ||
Comment 8•2 years ago
|
||
Updated•2 years ago
|
Pushed by evilpies@gmail.com: https://hg.mozilla.org/integration/autoland/rev/aa67c5a8e93c Fix clang-tidy build error
Comment 10•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f50502328881
https://hg.mozilla.org/mozilla-central/rev/b486e4108902
https://hg.mozilla.org/mozilla-central/rev/aa67c5a8e93c
Comment 11•2 years ago
|
||
Hey, it's great to see structuredClone()
land in Firefox!
For the record, the failing WPT tests in html/webappapis/structured-clone/structured-clone.any.html
are due to a bug I introduced in https://github.com/web-platform-tests/wpt/pull/30824, which was specific to the self.structuredClone()
runner and worked fine with other uses of the structured clone algorithm. This is now being fixed in https://github.com/web-platform-tests/wpt/pull/31034.
Updated•2 years ago
|
Description
•