Closed Bug 1543997 Opened 6 years ago Closed 6 years ago

Move default free op from JSRuntime to JSContext

Categories

(Core :: JavaScript: GC, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

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.

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
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: