Closed Bug 932384 Opened 10 years ago Closed 8 years ago

Rooting analysis needs to know about new PersistentRooted type

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 957891

People

(Reporter: jimb, Unassigned)

Details

Bug 892643 adds a new root type, PersistentRooted. It is okay to have a PersistentRooted on the stack across a GC. The rooting analysis needs to be updated before we can land any uses of PersistentRooted.
I'm guessing that this got resolved somehow, given that PersistentRooted has landed?
Flags: needinfo?(sphink)
Flags: needinfo?(jimb)
I, at least, don't know of any reason this bug needs to be kept open. I think it was filed mostly for Steve's benefit.
Flags: needinfo?(jimb)
Sorry, I really need to get a handle on my needinfos. I'm ignoring some straightforward ones like this because my queue is cluttered up with ancient really hard ones that I should track some other way.

annotations.js has this:

    return name.startsWith('Rooted') || name.startsWith('PersistentRooted');

so this is done. In fact, this bug is a dupe.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(sphink)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.