Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rebloor Unrelated to what this pull request is meant to fix, it seems worth filing it as a separate issue, but I did notice it while I was looking to this diff and so I thought to mention it to you here in the meantime:
"devtools" API doesn't technically exist in Chromium, and in Firefox isn't mandatory, it is implicitly prompted (and granted) at install time if the extension has a "devtools_page" property in the manifest.json file (unless listed as an optional permission in Firefox >= 77)
in Firefox >= 77 "devtools" permission can also be listed as an optional permission, if listed as an optional permission in the manifest.json file it will not be prompted and granted at install time and the extensions is going to be responsible of requesting it using the permissions WebExtensions API
It may be worth to mention that the "devtools" API namespace is only available in the DevTools Extension Pages (the devtools_page and the extension devtools panels created from the devtools_page using the
devtools.panels.create
anddevtools.panels.elements.createSidebarPane
), no other extension pages (e.g. background page or action popup panels pages) can access this API namespaceThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @rpl, I have created #21059 and will look into this shortly