Closed Bug 1621338 Opened 4 months ago Closed 29 days ago

Use the TargetList for listening to and retrieving worker targets

Categories

(DevTools :: Debugger, enhancement, P2)

enhancement

Tracking

(Fission Milestone:M6c, firefox79 fixed)

RESOLVED FIXED
Firefox 79
Fission Milestone M6c
Tracking Status
firefox79 --- fixed

People

(Reporter: ochameau, Assigned: jlast)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

(Whiteboard: dt-fission-m2-mvp)

Attachments

(1 file, 1 obsolete file)

Bug 1619622 focused on the process targets, but we would benefit from doing similar refactoring against worker targets:
https://searchfox.org/mozilla-central/rev/070a000dd49aac4a26147e137efcd91a728d13b8/devtools/client/debugger/src/client/firefox/targets.js#77-134
i.e. use TargetList.getAllTargets instead of listAllWorkerTargets, listServiceWorkerRegistration, ...

This work depends on bug 1594754, which adds service worker support for the browser toolbox and also bug 1621337 which is a followup to also expose service worker for the content toolbox codepath.

Blocks: 1626655
Priority: -- → P2
Whiteboard: dt-fission-m2-mvp

Tracking Fission DevTools bugs for Fission Nightly (M6) milestone

Fission Milestone: --- → M6
Depends on: 1621337
Assignee: nobody → jlaster
Status: NEW → ASSIGNED

Tracking dt-fission-m2-mvp bugs for Fission Nightly milestone (M6c)

Fission Milestone: M6 → M6c
No longer depends on: 1621337
Attachment #9146510 - Attachment description: Bug 1621338 - Use the TargetList for listening to and retrieving worker targets. r=ochameau → Bug 1621338 - Use the TargetList for listening to and retrieving worker targets. r=jdescottes
Depends on: 1641490
Attachment #9146487 - Attachment is obsolete: true
Pushed by jlaster@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/678264dd8b97
Use the TargetList for listening to and retrieving worker targets. r=jdescottes
Regressions: 1643122
Status: ASSIGNED → RESOLVED
Closed: 29 days ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 79
Regressions: 1646739
You need to log in before you can comment on or make changes to this bug.