Closed
Bug 1604514
Opened 5 years ago
Closed 5 years ago
Service Workers: zombie breakpoint
Categories
(DevTools :: Debugger, defect, P2)
DevTools
Debugger
Tracking
(firefox74 fixed)
RESOLVED
FIXED
Firefox 74
Tracking | Status | |
---|---|---|
firefox74 | --- | fixed |
People
(Reporter: Honza, Assigned: bhackett1024)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
STR:
- Open DevTools Toolbox, select the Debugger panel
- Load this page: http://janodvarko.cz/tests/debugger/service-workers/
- Click Register Worker in test case #1
- Create BP in service-worker.js on line 33
console.log("install", event)
- Click Unregister Worker in test case #1
- Click Register Worker in test case #1
- The debugger pauses on line 33, but the BP isn't displayed (the blue marker isn't there)
Honza
Reporter | ||
Updated•5 years ago
|
Blocks: dbg-worker-m2
Priority: P3 → P2
Assignee | ||
Comment 1•5 years ago
|
||
The problem here is related to how we treat service worker sources separately from one another, so that we can have multiple tabs with different versions of the source for different workers. When we unregister and re-register the service worker, the debugger client has a new source, and the pending breakpoints logic has some logic that prevents the breakpoint from being applied to the new service worker source. This logic is pretty weird and I tried removing it in bug 1534786, but it caused jest failures so I left it alone. Those jest failures don't seem to be happening anymore so hopefully we can just simplify the pending breakpoints logic and fix this bug in the meantime.
Assignee | ||
Comment 2•5 years ago
|
||
Updated•5 years ago
|
Assignee: nobody → bhackett1024
Status: NEW → ASSIGNED
Pushed by bhackett@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4ffb6c6fcbc0
Remove special case when applying pending breakpoints, r=loganfsmyth.
Comment 4•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox74:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 74
Updated•5 years ago
|
QA Whiteboard: [qa-74b-p2]
You need to log in
before you can comment on or make changes to this bug.
Description
•