Closed Bug 1225406 Opened 9 years ago Closed 8 years ago

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

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: lth, Unassigned)

References

(Blocks 1 open bug)

Details

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]
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
Depends on: 1225039
(Disallowing futexWait on the main thread will also require a number of test cases to be written and rewritten.)
Depends on: 1231333
Depends on: 1231335
Depends on: 1231337
Depends on: 1231338
Depends on: 1232973
No longer depends on: 1225039
No longer depends on: 1232973
Will ride with FF46 when that branches on Jan 25.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 1241776
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
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
To clarify, is this now enabled by default starting from Firefox 46?
Flags: needinfo?(lhansen)
(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 → ---
Depends on: 1309861
Depends on: 1312446
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
Closed: 9 years ago → 8 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.