Closed Bug 1654574 Opened 4 years ago Closed 4 years ago

Enable wasm in safe mode

Categories

(Core :: JavaScript Engine, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla80
Tracking Status
firefox80 --- fixed

People

(Reporter: jorendorff, Assigned: jorendorff)

References

Details

Attachments

(1 file)

In bug 1649117, it was observed that safe mode makes some websites extremely slow. Part of this might be due to wasm being disabled in safe mode.

In comment 12, jorendorff wrote:

"Safe mode" is a bad term for this—I don't believe it was never intended as a "stricter privacy" mode or a "heightened security for when you want to visit evil.com" mode, but rather as a troubleshooting mode, a "less weird stuff going on in my browser" mode.

Wasm is a feature of the Web platform. Turning it off is like turning off Promise or onclick events. Seems like, on net, that's more weird stuff, not less.

In comment 16, luke wrote:

Yes, agreed with Jason's comment; we shouldn't disable wasm with safe mode. If anything, disabling wasm makes more weird stuff happen, working against the goal of safe mode. To minimize magic, I think it'd make sense to disable tiering. Also, I haven't observed wasm-ion to be any less stable than wasm-baseline (perhaps even the opposite), so, given the perf advantage of the latter, I would recommend that safe mode only disables javascript.options.wasm_baselinejit.

Assignee: nobody → jorendorff
Status: NEW → ASSIGNED
Severity: -- → N/A
Type: task → enhancement
Priority: -- → P1
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: