Closed Bug 1644870 Opened 5 years ago Closed 5 years ago

Remove removeEventsTopTarget method from debugger codebase

Categories

(DevTools :: Debugger, task, P3)

task

Tracking

(firefox79 fixed)

RESOLVED FIXED
Firefox 79
Tracking Status
firefox79 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

Details

Attachments

(1 file)

See comment at https://phabricator.services.mozilla.com/D78562#inline-452905

The code is completely asymmetrical at the moment. We call addThreadEventListeners from onTargetAvailable -> addTarget -> attachThread -> attachTarget -> addThreadEventListeners. But we call removeThreadEventListeners from onTargetDestroyed -> removeEventsTopTarget -> removeThreadEventListeners

Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/53831248585f Remove removeEventsTopTarget method from debugger codebase r=jlast
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 79
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: