Closed Bug 1592158 Opened 5 years ago Closed 4 years ago

[jsdbg2] Remove `Debugger::hasAnyLiveFrames` and its vile henchmen

Categories

(Core :: JavaScript Engine, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: jimb, Unassigned)

References

Details

Bug 1592116 and bug 1592155 should leave Debugger::hasAnyLiveFrames with no interesting content, simply returning false in all cases. Deleting that leaves DebugAPI::markIteratively without any effects, so it can be deleted as well, and its single call site in js::gc::GCRuntime::markWeakReferences removed.

Priority: -- → P2

I dropped markIteratively in https://bugzilla.mozilla.org/show_bug.cgi?id=1592116 since https://bugzilla.mozilla.org/show_bug.cgi?id=1592155 had landed and there was no reason to leave it in there doing nothing. Hooray!

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED

[raises a pint]

You need to log in before you can comment on or make changes to this bug.