Closed
Bug 1518486
Opened 6 years ago
Closed 6 years ago
Disable extension devtools_page for private tabs when not allowed by users
Categories
(WebExtensions :: Developer Tools, enhancement, P1)
WebExtensions
Developer Tools
Tracking
(firefox67 verified)
Tracking | Status | |
---|---|---|
firefox67 | --- | verified |
People
(Reporter: rpl, Assigned: rpl)
References
(Blocks 1 open bug)
Details
Attachments
(4 files)
The devtools_page is the invisible extension page that is created for the devtools toolbox related to local tabs, and it is the entrypoint for any usage of the devtools APIs (as the entire devtools API namespace is never available in non-devtools extension pages).
If the tab related to the toolbox is a private tab, the extensions devtools_page should only be created for extensions that are allowed to access private browsing windows.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → lgreco
Blocks: webext-incognito
Status: NEW → ASSIGNED
Iteration: --- → 66.3 - Jan 7 - 20
Assignee | ||
Updated•6 years ago
|
Priority: -- → P1
Assignee | ||
Comment 1•6 years ago
|
||
This patch moves some "devtools related" test helpers into a shared support file,
and updates all the existing devtools API mochitests to use the new shared test helpers.
This change has also the following nice side-effects:
- remove any usage of the custom devtools module loader from the webextensions mochitests.
- use a custom "test blank toolbox panel" in all the devtools API tests that doesn't need a specific
panel to be selected (which should help to make the test able to run a bit faster because none of
the more expensive devtools panels is going to be loaded during the tests if not actually needed).
Assignee | ||
Comment 2•6 years ago
|
||
This patch applies the following changes:
- make sure that the WebExtensions internals do not create any devtools_page related to a private tabs' toolbox
if the extension is not allowed to access the private browsing windows
- define a new mochitest to verify the expended behaviors on the private windows when the extension is
allowed and not allowed
Depends on D15948
Assignee | ||
Comment 3•6 years ago
|
||
Depends on D15949
Pushed by luca.greco@alcacoop.it:
https://hg.mozilla.org/integration/autoland/rev/f5b40e3836f1
Move extension "devtools related" test helpers into a shared head_devtools.js support file. r=mixedpuppy
https://hg.mozilla.org/integration/autoland/rev/c63e693b1ce2
Disable extension devtools_page for private tabs when not allowed by users. r=mixedpuppy
https://hg.mozilla.org/integration/autoland/rev/5781d37b8a3b
Reuse shared loadTestSubscript test helper in other extensions mochitest-browser test files. r=mixedpuppy
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f5b40e3836f1
https://hg.mozilla.org/mozilla-central/rev/c63e693b1ce2
https://hg.mozilla.org/mozilla-central/rev/5781d37b8a3b
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
This issue is verified as fixed on Firefox 67.0a1(20190204214259) under Win 7 64-bit and Mac OS X 10.14.1.
Please see the attached video.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•