1795452 - Implement Array.fromAsync

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Comment 1

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1795452 - Expose CreateAsyncFromSyncIter to self-hosted code r?arai

Comment 2

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1795452 - Add the three-argument GetIterator function. r?arai

Depends on D159640

Comment 3

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1795452 - Initial implementation of Array.fromAsync r?arai

Depends on D159641

Comment 4

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1795452 - Add shell option to enable/disable Array.fromAsync r?arai

Depends on D159642

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/aef9d287cced
Expose CreateAsyncFromSyncIter to self-hosted code r=arai
https://hg.mozilla.org/integration/autoland/rev/ffffdf0dad7b
Add the three-argument GetIterator function. r=arai
https://hg.mozilla.org/integration/autoland/rev/55ab67c54574
Initial implementation of Array.fromAsync r=arai
https://hg.mozilla.org/integration/autoland/rev/021312205c09
Add shell option to enable/disable Array.fromAsync r=arai

Comment 6

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/aef9d287cced
https://hg.mozilla.org/mozilla-central/rev/ffffdf0dad7b
https://hg.mozilla.org/mozilla-central/rev/55ab67c54574
https://hg.mozilla.org/mozilla-central/rev/021312205c09

Comment 7

[Hamish Willee]

For the purposes of MDN documentation:

1. Am I correct that this is only enabled on Nightly (initially 108?), and that there is no preference value to enable it in release?
2. Do we know of other implementations in progress?
3. Is this "ready" to document? Specifically the explainer seems to indicate that this is TC2. Sometimes we document at TC3, but normally only if the engineering team (you!) tell us that the functionality is likely to be stable and want it - this is to avoid churn while API names and behaviour are still in flux.

Comment 8

[Matthew Gaudet (he/him) [:mgaudet]]

1. Currently this is disabled by default, even in nightly. The pref is only available in nightly.
2. I'm not aware of any other in progress implementations: v8's bug is here, with no activity; couldn't find one for JSC.
3. The explainer is out of date, as the proposal hit Stage 3 in September.

Personally, I'd hold off on documenting this until we've given an attempt to ship it on Nightly to discover any name conflicts / webcompat issues.

Comment 9

[Hamish Willee]

FF108 docs for this on hold as suggested ^^^- thanks Matthew!

Tracking issue on MDN is: https://github.com/mdn/content/issues/22145.

I'll remove "dev-doc-needed" for this bug. We'll keep an eye on the meta bug and the shipping bug; but feel free to ping us on the above issue if this is shipped on nightly, or otherwise is ready to document.

Comment 10

[Hamish Willee]

FYI only, most of the stage 3 array-from-async docs work for this was done in https://github.com/mdn/content/pull/25775, following the feature shipping in Safari.