Closed Bug 1588050 Opened 5 years ago Closed 5 years ago

Switch ON the Omniscient Browser Toolbox by default

Categories

(DevTools :: Framework, task, P1)

task

Tracking

(Fission Milestone:M6, relnote-firefox 73+, firefox73 fixed)

RESOLVED FIXED
Firefox 73
Fission Milestone M6
Tracking Status
relnote-firefox --- 73+
firefox73 --- fixed

People

(Reporter: pbro, Assigned: ochameau)

References

(Blocks 4 open bugs)

Details

(Whiteboard: dt-fission-m1)

Attachments

(1 file, 4 obsolete files)

The work on supporting Fission in DevTools started with enabling the Browser Toolbox to inspect/debug multiple processes.

This work is happening behind the pref devtools.browsertoolbox.fission (defined here).

Once this work is complete, we should enable the pref so the new "Omniscient" Browser Toolbox can be used by default everywhere.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: General → Framework
Assignee: nobody → gl
Status: NEW → ASSIGNED
Priority: P2 → P1
Depends on: 1591993
Blocks: 1593753
Blocks: 1571632
Depends on: 1594507
Depends on: 1576115
Depends on: 1599037
Summary: Switch ON the Omnicient Browser Toolbox by default → Switch ON the Omniscient Browser Toolbox by default
Assignee: gl → poirot.alex

Tracking for Fission Nightly (M6)

Whiteboard: dt-fission → dt-fission dt-fission-m1

Tracking for Fission Nightly (M6)

Fission Milestone: --- → M6

Trying to list the small tweaks we might want to do before enabling:

  • remove emojis in the title
  • localize settings checkbox
  • add a learn more link next to the settings checkbox, instead of the warning WIP warning (need to have a page to link to however)
  • maybe rename the "Main Process" target in about:debugging (but I don't have a good name)
Depends on: 1603361
Depends on: 1603366
Whiteboard: dt-fission dt-fission-m1 → dt-fission-m1
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/58075bcbdcef Turn the Multiprocess Browser Toolbox on by default on Nightly. r=jdescottes
Flags: needinfo?(poirot.alex)

For browser_aboutdebugging_process_main.js , the test is checking the text of the Main Process item.
I think it's fair to add await pushPref("devtools.browsertoolbox.fission", false); at the beginning of the test for this one.

Attachment #9116663 - Attachment is obsolete: true
Attached file Bug 1588050 - Fix about:debugging test. (obsolete) (deleted) —
Attached file Bug 1588050 - Fix browser console test. (obsolete) (deleted) —

Ignore worker targets in the console as we workerTarget.getFront("console") fails.
It relates to bug 1592584 and properly support workers by using a real console actor
rather than piping the console message via platform API to the main thread.

Attachment #9116676 - Attachment is obsolete: true
Attachment #9116677 - Attachment is obsolete: true
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/721fefd09f16 Turn the Multiprocess Browser Toolbox on by default on Nightly. r=jdescottes
Blocks: 1605036
Blocks: 1605046
Regressions: 1605152
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 73
Regressions: 1605107
Regressions: 1605347
Attachment #9103730 - Attachment is obsolete: true
Regressions: 1605282
Regressions: 1604751
Flags: needinfo?(poirot.alex)

Added to the final Fx73 relnotes.

Blocks: 1625937
Blocks: 1625939
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: