Switch ON the Omniscient Browser Toolbox by default
Categories
(DevTools :: Framework, task, P1)
Tracking
(Fission Milestone:M6, 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)
(deleted),
text/x-phabricator-request
|
Details |
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.
Comment 1•5 years ago
|
||
Bugbug thinks this bug should belong to this component, but please revert this change in case of error.
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 2•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Comment 4•5 years ago
|
||
Tracking for Fission Nightly (M6)
Reporter | ||
Updated•5 years ago
|
Comment 6•5 years ago
|
||
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)
Assignee | ||
Comment 7•5 years ago
|
||
Updated•5 years ago
|
Comment 9•5 years ago
|
||
Backed out for dt failures on browser_aboutdebugging_process_main.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/660a95371114ac87ed1f171513cfaa3ca4aab9c8
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=281704454&repo=autoland&lineNumber=3963
There were also dt failures on browser_dbg-browser-content-toolbox.js
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=281702993&repo=autoland&lineNumber=3909
Comment 10•5 years ago
|
||
Comment 11•5 years ago
|
||
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.
Updated•5 years ago
|
Assignee | ||
Comment 12•5 years ago
|
||
Assignee | ||
Comment 13•5 years ago
|
||
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.
Updated•5 years ago
|
Updated•5 years ago
|
Comment 14•5 years ago
|
||
Comment 15•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Description
•