Ship ArrayBuffer.prototype.transfer proposal
Categories
(Core :: JavaScript Engine, task, P3)
Tracking
()
People
(Reporter: mgaudet, Assigned: mgaudet)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(4 files)
Assignee | ||
Comment 1•1 year ago
|
||
Andre: Is there anything you see blocking us from just unflagging this now?
Comment 2•1 year ago
|
||
I think it should be okay to ship the implementation.
Updated•1 year ago
|
Assignee | ||
Comment 3•1 year ago
|
||
Updated•1 year ago
|
Assignee | ||
Comment 4•1 year ago
|
||
Depends on D195572
Assignee | ||
Comment 5•1 year ago
|
||
Update the jstests skip list for SharedArrayBuffer tests as well.
Assignee | ||
Comment 6•1 year ago
|
||
Special handling is required for the two transfer methods, as they detach the
array buffer, which breaks future tests. This patch adds gStatefulProperties
which are those that cannot be invoked without changing object state in such a
way as to break the test harness.
Depends on D195574
Updated•1 year ago
|
Assignee | ||
Comment 7•1 year ago
|
||
Sent Intent to Ship: https://groups.google.com/a/mozilla.org/g/dev-platform/c/LXFQsMupYYY
Updated•1 year ago
|
Comment 9•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f89fa88d7606
https://hg.mozilla.org/mozilla-central/rev/b5909f5010c7
https://hg.mozilla.org/mozilla-central/rev/7eaa2d12be9f
https://hg.mozilla.org/mozilla-central/rev/eff46450e868
Comment 10•1 year ago
|
||
:mgaudet could you consider nominating this for a release note? (Process info)
Assignee | ||
Comment 11•1 year ago
|
||
Release Note Request (optional, but appreciated)
[Why is this notable]: A new web platform feature
[Affects Firefox for Android]: Yes
[Suggested wording]: "Enabled the ArrayBuffer.prototype.transfer proposal methods, which enables transferring ownership of ArrayBuffer data."
[Links (documentation, blog post, etc)]:
Comment 12•1 year ago
|
||
Thanks, added to the Fx122 nightly release notes, please allow 30 minutes for the site to update.
Keeping the relnote-firefox flag as ? to keep it on the radar for inclusion in the final Fx122 release notes
Comment 13•1 year ago
|
||
:mgaudet can you confirm the expected state of this? Wondering if it in fact enabled for Fx122.
https://hg.mozilla.org/mozilla-central/diff/f89fa88d7606eabe5797e9bb9a058f151c16f637/modules/libpref/init/StaticPrefList.yaml
The pref value is false.
Assignee | ||
Comment 14•1 year ago
|
||
So that patch merely made the pref flip possible; the flip actually happened in the next patch (https://hg.mozilla.org/mozilla-central/rev/b5909f5010c7e45247034c8147e0cc5d16f92400).
Assignee | ||
Comment 15•1 year ago
|
||
(You can also check in console with (new ArrayBuffer(10)).transfer()
)
Comment 16•1 year ago
|
||
Sorry for the noise, mistake on my part
Updated•1 year ago
|
Comment 17•1 year ago
|
||
FF122 MDN docs work for this done in https://github.com/mdn/content/issues/31101. Just a release note, removing markup to indicate "experimental", and update browser compatibility data.
Description
•