Closed
Bug 1487836
Opened 6 years ago
Closed 6 years ago
Fix innerWindowID used for the WebExtensions devtools API eval denied errors
Categories
(WebExtensions :: Developer Tools, enhancement)
WebExtensions
Developer Tools
Tracking
(firefox63 verified)
VERIFIED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | verified |
People
(Reporter: rpl, Assigned: rpl)
Details
Attachments
(1 file)
Bug 1476145 we recently removed a number of calls to "getInterface(nsIDOMWindowUtils)" that are not necessary anymore, unfortunately during the mass rewrite we assigned the windowUtils to the var innerWindowId by mistake: https://hg.mozilla.org/mozilla-central/diff/640fd7808af5/devtools/server/actors/addon/webextension-inspected-window.js the warning message is still logged and visible in the browser console, but it not being made visible in the webconsole related to the tab which the warnings is related to.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → lgreco
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9005679 [details] Bug 1487836 - Fix innerWindowID used for the WebExtensions devtools API eval denied errors. Julian Descottes [:jdescottes][:julian] has approved the revision.
Attachment #9005679 -
Flags: review+
Pushed by luca.greco@alcacoop.it: https://hg.mozilla.org/integration/autoland/rev/55570b34ff7c Fix innerWindowID used for the WebExtensions devtools API eval denied errors. r=jdescottes
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/55570b34ff7c
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Comment 5•6 years ago
|
||
Is manual testing required on this bug? If yes, please provide some STR and the proper extension(if required) or set the “qe-verify -“ flag. Thanks!
Flags: needinfo?(lgreco)
Assignee | ||
Comment 6•6 years ago
|
||
STR: - open a new tab and navigate to https://addons.mozilla.org/en-US/firefox/addon/react-devtools - install the react devtools addon - open the webconsole panel from the developer tools for the "addons.mozilla.org" tab (e.g. from the hamburger menu -> "Web Developer" -> "Web Console") Expected Behavior on the fixed build: - An error message "The extension "React Developer Tools" is not allowed to access https://addons.mozilla.org/en-US/firefox/addon/react-devtools/..." should be logged in the tab's webconsole Actual Behavior on the affected builds: - No error message ""The extension "React Developer Tools" is not allowed to access ..." in the tab's webconsole - The error message has been logged to the "Browser Console" (e.g. from the hamburger menu -> "Web Developer" -> "Browser Console")
Flags: needinfo?(lgreco)
Comment 7•6 years ago
|
||
Thank you Luca for clarifying the steps. Verified that the issue is no longer reproducible using the latest build of Nightly (07/09/2018) running on Windows 10 x64 and Ubuntu 18.04, both consoles (tab and browser) are showing the error: "The extension "React Developer Tools" is not allowed to access ...".
Updated•6 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•