In the browser toolbox's context selector, display frame targets under their related content process target
Categories
(DevTools :: Framework, enhancement, P3)
Tracking
(Fission Milestone:Future, firefox102 fixed)
Tracking | Status | |
---|---|---|
firefox102 | --- | fixed |
People
(Reporter: ochameau, Assigned: nchevobbe)
References
(Blocks 2 open bugs)
Details
(Whiteboard: dt-fission-future)
Attachments
(2 files)
Today, we display targets sorted by type, but it would be really handy in the context of the browser toolbox to display frame targets under each content process.
It would actually help understanding what are each content process.
I could swear we already had a bug for this?
In order to do that, we probably should expose the processID of all frame targets via their form:
https://searchfox.org/mozilla-central/rev/85ae3b911d5fcabd38ef315725df32e25edef83b/devtools/server/actors/targets/browsing-context.js#563-565
https://searchfox.org/mozilla-central/rev/85ae3b911d5fcabd38ef315725df32e25edef83b/devtools/client/fronts/targets/browsing-context.js#31-33
Given this additional attribute on each targetFront for processes, we can probably just tweak the UI to better sort the targets:
https://searchfox.org/mozilla-central/rev/85ae3b911d5fcabd38ef315725df32e25edef83b/devtools/client/webconsole/components/Input/EvaluationContextSelector.js#107-147
We might have to introduce a sorting specific to the browser toolbox?
Reporter | ||
Comment 1•4 years ago
|
||
Reporter | ||
Updated•4 years ago
|
Comment 2•4 years ago
|
||
Alex, there's a patch here but no reviewers. Could you please finalize this to close the bug?
Also, there is no dt-xx whiteboard tag for this, do we need one for dt-m2 or dt-m3?
Comment 3•4 years ago
|
||
Honza, this DevTools Fission bug has a patch, but no reviewers. Should this bug block shipping Fission? It blocks dt-fission-execution-context meta bug 1605153, but doesn't have a dt-fission
whiteboard tag.
Comment 4•4 years ago
|
||
Thanks for the ping Chris, flags updated.
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Comment 5•4 years ago
|
||
The attached patch was just a demo, I would need more time, which I don't have to finish this.
Updated•4 years ago
|
Reporter | ||
Updated•4 years ago
|
Comment 6•4 years ago
|
||
Honza said this is not part of dt-fission-m3-mvp and is not considered Fission M7 blocker.
Comment 7•4 years ago
|
||
Moving "dt-fission-m3-reserve" bugs to "dt-fission-future" because they don't block Fission MVP.
Assignee | ||
Comment 8•2 years ago
|
||
Depends on D145880
Updated•2 years ago
|
Comment 10•2 years ago
|
||
bugherder |
Description
•