Closed
Bug 1642907
Opened 4 years ago
Closed 4 years ago
AboutDebugging should not modify the form of webextension descriptors
Categories
(DevTools :: about:debugging, task, P3)
DevTools
about:debugging
Tracking
(firefox79 fixed)
RESOLVED
FIXED
Firefox 79
| Tracking | Status | |
|---|---|---|
| firefox79 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
Details
Attachments
(2 files)
about:debugging currently modifies the WebExtension descriptors it retrieves in order to remove the "manifestURL" property. The reason we do that is to avoid showing the link in the about:debugging UI when debugging remote runtimes.
But this is hacky, and should be handled at the component level rather than by modifying an external object.
|
Assignee | |
Comment 1•4 years ago
|
||
|
||
Comment 2•4 years ago
|
||
Bugbug thinks this bug should belong to this component, but please revert this change in case of error.
Component: General → about:debugging
|
|
Assignee | |
Comment 3•4 years ago
|
||
Depends on D78014
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0cd09611ce60 Stop modifying descriptor form in aboutdebugging r=daisuke https://hg.mozilla.org/integration/autoland/rev/7fc7cf6b3de2 Use dictionary argument in RuntimePage::renderDebugTargetPane r=daisuke
|
||
Comment 5•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/0cd09611ce60
https://hg.mozilla.org/mozilla-central/rev/7fc7cf6b3de2
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox79:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 79
You need to log in
before you can comment on or make changes to this bug.
Description
•