Closed
Bug 1592158
Opened 5 years ago
Closed 5 years ago
[jsdbg2] Remove `Debugger::hasAnyLiveFrames` and its vile henchmen
Categories
(Core :: JavaScript Engine, enhancement, P2)
Core
JavaScript Engine
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.
|
||
Updated•5 years ago
|
Priority: -- → P2
|
||
Comment 1•5 years ago
|
||
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: 5 years ago
Resolution: --- → FIXED
|
Reporter | |
Comment 2•5 years ago
|
||
[raises a pint]
You need to log in
before you can comment on or make changes to this bug.
Description
•