Closed Bug 1721733 Opened 3 years ago Closed 3 years ago

browser_target_list_service_workers_navigation.js fails with server side target switching enabled

Categories

(DevTools :: Framework, defect)

defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1691585

People

(Reporter: ochameau, Assigned: nchevobbe)

References

Details

Attachments

(1 obsolete file)

This test fails because the TargetCommand destroy all the targets on top level target switching:
https://searchfox.org/mozilla-central/rev/77256682a1fe0e73c5ad34c593b1ecacfefdf4e1/devtools/shared/commands/target/target-command.js#168-174

Unfortunately, fixing that isn't enough to make the test pass.
The SW target seems to not be destroyed when we unregister it...

On target switch we destroy all targets currently registered in TargetCommand,
this includes Service Worker targets.
Whereas we expect them to be kept across target switch, and only
destroyed when the LegacyListener tells us they are destroyed.

Fixing this, seem to make the test fail later, or there is yet something else.
The SW target isn't destroyed when we unregister the .org SW.

Whiteboard: dt-fission-m3-triage
Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED

the test run fine after I apply the patch I pushed for Bug 1691585 (https://phabricator.services.mozilla.com/D120710), so I'm going to close this as a duplicate

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
Whiteboard: dt-fission-m3-triage
Attachment #9232508 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: