Closed Bug 1808574 Opened 1 year ago Closed 1 year ago

Migrate old worker test to about:debugging test

Categories

(DevTools :: about:debugging, task)

task

Tracking

(firefox110 fixed)

RESOLVED FIXED
110 Branch
Tracking Status
firefox110 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

(Blocks 2 open bugs, Regressed 1 open bug)

Details

Attachments

(1 file)

The following tests:
https://searchfox.org/mozilla-central/search?q=session_id&path=worker-console&case=false&regexp=false
are all spawning toolbox for regular worker via gDevTools.showToolbox.
This isn't something that happen in production.
First, we no longer support debugging regular workers via dedicated toolboxes.
We only support spawning dedicated toolboxes for service workers and parent process privileged workers.
Then, we aren't using gDevTools.showToolbox for these toolboxes, but open an about:devtools-toolbox URL from about:debugging.

These tests should be migrated to about:debugging tests.

This will help remove a few callsites still spawning toolboxes for regular workers.
Only service workers and chrome/parent process workers can be debugged independantly,
in a dedicated toolbox. That, only from about:debugging, by opening a toolbox in a tab.

Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED

The Bugbug bot thinks this bug should belong to the 'DevTools::about:debugging' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → about:debugging
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6913e09031db
[devtools] Migrate old worker test to about:debugging test. r=jdescottes
Regressions: 1808835
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: