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

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
4 years ago
2 years ago

People

(Reporter: lth, Unassigned)

Tracking

(Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Reporter

Description

4 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

4 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

4 years ago
Depends on: 1225039
Reporter

Comment 2

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

Updated

4 years ago
Depends on: 1231333
Reporter

Updated

4 years ago
Depends on: 1231335
Reporter

Updated

4 years ago
Depends on: 1231337
Reporter

Updated

4 years ago
Depends on: 1231338
Reporter

Updated

4 years ago
Depends on: 1232973
Reporter

Updated

3 years ago
No longer depends on: 1225039
Reporter

Updated

3 years ago
No longer depends on: 1232973
Reporter

Comment 3

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

Updated

3 years ago
Depends on: 1241776
Reporter

Comment 4

3 years 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

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

Comment 6

3 years 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

3 years ago
Depends on: 1309861
Reporter

Updated

3 years ago
Depends on: 1312446
Reporter

Comment 7

2 years 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: 3 years ago2 years 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.