1580080 - Simplify GCRuntime::findDeadCompartments by using a compartment to compartment map

Status: RESOLVED INVALID

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

Description

[Jon Coppeard (:jonco)]

This traverses all compartment to compartment edges for CCWs and debugger weak map entries. Currently this happens as part of its main loop. It would be simpler to build a map of these edges at the start of the method and then use it in the main loop. This is closer to what happens when we calculate sweep groups, which is similar.

Comment 1

[Jon Coppeard (:jonco)]

Attachment: Bug 1580080 - Use a compartment to compartment map when finding dead compartments r?sfink

This simplifies GCRuntime::findDeadCompartments and is the basis of future changes.

Depends on D45386

Comment 2

[Jon Coppeard (:jonco)]

This is also no longer needed since the realisation in bug 1580205 comment 3. We can simplify this by backing out bug 1577146 instead.