Move default free op from JSRuntime to JSContext

RESOLVED FIXED in Firefox 68

Status

()

task
P3
normal
RESOLVED FIXED
2 months ago
2 months ago

People

(Reporter: jonco, Assigned: jonco)

Tracking

Trunk
mozilla68
Points:
---

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

2 months ago

At the moment we have one default js::FreeOp per runtime. This can be used concurrently by parse threads, e.g. by AutoEnterAnalysis. Right now this doesn't touch any of the FreeOp's shared state and so is safe, but it's asking for trouble.

We should have a separate FreeOp per context and use that.

Comment 2

2 months ago
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/03b7cf8b0c18
Move default free op from JSRuntime to JSContext r=sfink
Type: defect → task

Comment 3

2 months ago
bugherder
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.