Remove "show content message" toggle and related code
Categories
(DevTools :: General, task)
Tracking
(firefox110 fixed)
| Tracking | Status | |
|---|---|---|
| firefox110 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Followup from https://phabricator.services.mozilla.com/D164699#inline-908627
Now that we only support the "fission browser toolbox" only, we can get rid of the "show content message" toggle and related code. The new way of filtering messages is now rather about restricting only to parent process.
https://searchfox.org/mozilla-central/rev/17aeb39742eba71e0936ae44a51a54197100166d/devtools/client/webconsole/actions/ui.js#54-65,
https://searchfox.org/mozilla-central/rev/17aeb39742eba71e0936ae44a51a54197100166d/devtools/client/webconsole/constants.js#45 (and its callsites)
https://searchfox.org/mozilla-central/rev/17aeb39742eba71e0936ae44a51a54197100166d/devtools/client/webconsole/constants.js#81-82
https://searchfox.org/mozilla-central/search?q=showContentMessages&path=&case=false®exp=false
and maybe other spots
|
Assignee | |
Comment 1•1 year ago
|
||
This was no longer implemented/shown in the fission browser toolbox.
Now that we removed the non-fission toolbox, we can remove this feature.
Instead, content process messages are shown/hidden depending on browser toolbox's scope.
"everything" will show them. "parent-process" won't.
|
||
Updated•1 year ago
|
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ff6030743bb4 [devtools] Remove code related to "show content message" toggle. r=perftest-reviewers,nchevobbe,kshampur
|
||
Comment 3•1 year ago
|
||
| bugherder | ||
Description
•