Closed Bug 1588302 Opened 10 months ago Closed 10 months ago

Refactor heap state a little

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(3 files)

The heap state should really live GCRuntime rather than JSRuntime, and its accessor functions should live in a GC header rather than jspubtd.h.

Assignee: nobody → jcoppeard
Priority: -- → P3

jspubtd.h is for type definitions so this moves the heap state functions to somewhere more suitable.

Depends on D49116

The heap state is GC data; it should live in GCRuntime.

Depends on D49117

Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d1fb75a00d99
Move heap state accessors to public/HeapAPI.h r=sfink
https://hg.mozilla.org/integration/autoland/rev/41320766c363
Move heap state to GCRuntime r=sfink
https://hg.mozilla.org/integration/autoland/rev/98a43588ff8a
Make AutoHeapSession use a GCRuntime pointer r=sfink
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.