[meta] Enable SharedArrayBuffer and Atomics on DevEd, Aurora, Beta, and Release

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
2 years ago
4 months ago

People

(Reporter: lth, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
The gating condition for letting SAB+Atomics ride the trains (up to, but not including, Release) is to get the spec to Stage 2 in TC39.  At the time of writing we're not there yet.

Not sure what needs to be done technically.  There's a condition in js/src/moz.build that currently constrains the feature to nightly; perhaps it can simply be generalized to include tests for all-but-release.
Keywords: dev-doc-needed
Whiteboard: [DocArea=JS]
(Reporter)

Comment 1

2 years ago
Update: After some discussion at Mozlando the decision is to let SAB+Atomics ride the trains - all the way - behind a pref, starting as soon as the code can be written.  The FF45 train departs mid-week next week, that may be too soon for the work that needs to be done and too risky to just cram it in.  So the FF46 train is the target.

As part of that work, a few work items will need to be completed and they will appear as dependencies on this bug:

- make the TypedArray implementation compliant with the SAB spec
- disallow futexWait blocking on the main thread, needs to be backed by a spec change
- double-check that all the feature gating is working correctly
- implement a suitable pref

My intent is to keep this available in Nightly even without the pref (ie the pref defaults to true in Nightly).  Please, somebody, scream if that is inappropriate.
Summary: Enable SharedArrayBuffer and Atomics on DevEd, Aurora, and Beta → [meta] Enable SharedArrayBuffer and Atomics on DevEd, Aurora, and Beta
(Reporter)

Updated

2 years ago
Depends on: 1225039
(Reporter)

Comment 2

2 years ago
(Disallowing futexWait on the main thread will also require a number of test cases to be written and rewritten.)
(Reporter)

Updated

2 years ago
Depends on: 1231333
(Reporter)

Updated

2 years ago
Depends on: 1231335
(Reporter)

Updated

2 years ago
Depends on: 1231337
(Reporter)

Updated

2 years ago
Depends on: 1231338
(Reporter)

Updated

2 years ago
Depends on: 1232973
(Reporter)

Updated

2 years ago
No longer depends on: 1225039
(Reporter)

Updated

2 years ago
No longer depends on: 1232973
(Reporter)

Comment 3

2 years ago
Will ride with FF46 when that branches on Jan 25.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
(Reporter)

Updated

a year ago
Depends on: 1241776
(Reporter)

Comment 4

a year ago
Reopening because there's another patch coming for the #define.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: [meta] Enable SharedArrayBuffer and Atomics on DevEd, Aurora, and Beta → [meta] Enable SharedArrayBuffer and Atomics on DevEd, Aurora, Beta, and Release
(Reporter)

Updated

a year ago
Status: REOPENED → RESOLVED
Last Resolved: 2 years agoa year ago
Resolution: --- → FIXED
To clarify, is this now enabled by default starting from Firefox 46?
Flags: needinfo?(lhansen)
(Reporter)

Comment 6

a year ago
(In reply to Marco Castelluccio [:marco] from comment #5)
> To clarify, is this now enabled by default starting from Firefox 46?

No.  It is only enabled by default on Nightly.

I should not have closed this bug, I was being careless while I was cleaning up.  I will reopen.
Status: RESOLVED → REOPENED
Flags: needinfo?(lhansen)
Resolution: FIXED → ---
(Reporter)

Updated

8 months ago
Depends on: 1309861
(Reporter)

Updated

8 months ago
Depends on: 1312446
Blocks: 1317856
(Reporter)

Comment 7

5 months ago
Closing this because the blockers are all closed, but note that we may back out the code so as not to enable this on Release.  Bug 1312446 for more information.
Status: REOPENED → RESOLVED
Last Resolved: a year ago5 months ago
Resolution: --- → FIXED
Removing ddn, bug 1312446 has the info on what is shipped when and also has ddn.
Keywords: dev-doc-needed
Whiteboard: [DocArea=JS]
You need to log in before you can comment on or make changes to this bug.