Expose a targetType attribute on all target actor forms
Categories
(DevTools :: Framework, enhancement)
Tracking
(firefox137 fixed)
| Tracking | Status | |
|---|---|---|
| firefox137 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
We have something a bit brittle in Target Command to define the targetType for each target front:
https://searchfox.org/mozilla-central/rev/548b6981501f59e3c9f2f7851c013e7d53c4e72f/devtools/shared/commands/target/target-command.js#711-741
It would be nice to convey the existing targetType attribute from server to client so that we can have a simpler and more reliable attribute to distinguish the target types.
This would also be used by VS Code.
|
Assignee | |
Comment 1•4 months ago
|
||
This will help distinguish targets by types in the frontend, as well as in Vs Code.
|
||
Updated•4 months ago
|
|
||
Comment 3•4 months ago
|
||
| bugherder | ||
|
|
Assignee | |
Comment 4•4 months ago
|
||
This will be useful when we start removing the backward compat code.
|
||
Comment 6•4 months ago
|
||
| bugherder | ||
|
||
Comment 7•4 months ago
|
||
A patch has been attached on this bug, which was already closed. Filing a separate bug will ensure better tracking. If this was not by mistake and further action is needed, please alert the appropriate party. (Or: if the patch doesn't change behavior -- e.g. landing a test case, or fixing a typo -- then feel free to disregard this message)
Description
•