Closed Bug 1332240 Opened 3 years ago Closed 3 years ago

Perma failure in TEST-UNEXPECTED-FAIL | js1_8_5/extensions/dataview.js | (args: "") when trunk moves to aurora 53


(Core :: JavaScript Engine, defect, blocker)

Not set



Tracking Status
firefox53 + fixed


(Reporter: cbook, Assigned: lth)





(2 files)

TEST-UNEXPECTED-FAIL | js1_8_5/extensions/dataview.js | (args: "")

[task 2017-01-19T09:46:48.198970Z] BUGNUMBER: 575688

[task 2017-01-19T09:46:48.198982Z] STATUS: DataView tests

[task 2017-01-19T09:46:48.199007Z] js1_8_5/extensions/dataview.js:57:6 ReferenceError: SharedArrayBuffer is not defined

[task 2017-01-19T09:46:48.199018Z] Stack:

[task 2017-01-19T09:46:48.199036Z]   bufferize@js1_8_5/extensions/dataview.js:57:6

[task 2017-01-19T09:46:48.199052Z]   test@js1_8_5/extensions/dataview.js:73:14

[task 2017-01-19T09:46:48.199068Z]   @js1_8_5/extensions/dataview.js:1652:1

[task 2017-01-19T09:46:48.199089Z] TEST-UNEXPECTED-FAIL | js1_8_5/extensions/dataview.js | (args: "")
[Tracking Requested - why for this release]:
note: requested blocking because this might cause tree closure after merge day , since this was found by uplift simulations for running trunk as aurora
The test is easily fixed but it's exceptionally troubling that this happens when trunk is uplifted as aurora, as this feature should be enabled on aurora.
This is a problem primarily with the test case and secondarily with the JS shell.  The problem is that the shell has its own notion of when shared memory and atomics are enabled by default, and that notion is "nightly only", which is not in sync with Firefox and hence the bug in the test case was exposed here.

This test would not have failed had it been run in the browser (though it would fail in beta :)
Add a guard to the test case to check for availability of SharedArrayBuffer.
Assignee: nobody → lhansen
Attachment #8828280 - Flags: review?(bbouvier)
Enable SAB+Atomics by default in the shell, they can be turned off using a shell argument.

I can't think of a reason why we should be tracking Firefox here, but we could do so, by changing the #ifdef instead of removing it as I do.  (SAB+Atomics are enabled by default in Firefox Nightly and Aurora builds only.)
Attachment #8828283 - Flags: review?(bbouvier)
Attachment #8828280 - Flags: review?(bbouvier) → review+
Comment on attachment 8828283 [details] [diff] [review]

Review of attachment 8828283 [details] [diff] [review]:

Yeah, I'd really prefer to be in sync with Firefox; istr fuzzers test beta/release shells too (?), and there might issues when patches are non being uplifted.

rs=me for doing that.
Attachment #8828283 - Flags: review?(bbouvier) → review+
Bug 1332240 - Enable SharedArrayBuffer and Atomics by default in JS shell also for Aurora. r=bbouvier
Tracking 53+ for this issue.
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.