Closed Bug 1557170 Opened 6 months ago Closed 3 months ago

Make Service Workers debugging compatible with remote debugging

Categories

(DevTools :: about:debugging, task, P2)

task

Tracking

(firefox70 fixed)

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: perry, Assigned: jdescottes)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

There's multiple Service Workers implementations (controlled by a pref), and the workers & their metadata (e.g. scope) may exist in difference processes depending on the implementation. Devtools logic must account for for these differences by reading the pref. Getting the pref via Device{Front,Actor}.getDescription(), however, causes test failures when tests are cleaning up.

Type: defect → task
Priority: -- → P2

Follow up to 1552945, can't be worked on before the other bug lands. See the conversation on the review at https://phabricator.services.mozilla.com/D31905?id=106857#inline-197492 for more context.
Basically there is a client-side check for swm.isParentInterceptEnabled(), and it needs to move to the server.

Component: General → about:debugging
Depends on: 1552945

Would be nice to get this in 70, now that the rest of the refactor has landed

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2e8bbe940e42
Add isParentInterceptEnabled trait to ServiceWorkerRegistration actor r=daisuke
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
You need to log in before you can comment on or make changes to this bug.