Closed
Bug 1745818
Opened 5 months ago
Closed 3 months ago
Treat host permissions as optional in MV3 manifest
Categories
(WebExtensions :: General, enhancement, P1)
WebExtensions
General
Tracking
(firefox100 fixed)
RESOLVED
FIXED
100 Branch
Tracking | Status | |
---|---|---|
firefox100 | --- | fixed |
People
(Reporter: zombie, Assigned: zombie)
References
(Blocks 3 open bugs)
Details
(Whiteboard: [mv3-m2])
Attachments
(1 file)
- ⬜ When a manifest v3 extension is installed, don’t add host_permissions and content_script match patterns to allowedOrigins.
- ⬜ Remove host permissions from the initial install prompt.
- ⬜ Implement granted_host_permissions manifest key to revert to previous behavior for testing and privileged extensions.
- ⬜ Implement a pref for manifest v2 extensions to behave the same, default to false.
- ⬜ Adjust our existing mv3 tests to work with these changes.
- ⬜ Write new tests for all of the above.
Updated•5 months ago
|
Updated•5 months ago
|
Points: --- → 3
Assignee | ||
Updated•5 months ago
|
Points: 3 → 5
Assignee | ||
Updated•5 months ago
|
Assignee: nobody → tomica
Severity: -- → N/A
Priority: -- → P1
Assignee | ||
Comment 1•4 months ago
|
||
Updated•4 months ago
|
Attachment #9259523 -
Attachment description: WIP: Bug 1745818 - Treat host_permissions as optional in mv3 → Bug 1745818 - Treat host_permissions as optional in mv3, r=robwu
Updated•3 months ago
|
Attachment #9259523 -
Attachment description: Bug 1745818 - Treat host_permissions as optional in mv3, r=robwu → Bug 1745818 - Treat host permissions as optional in mv3, r=robwu
Pushed by tjovanovic@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/53b7b2117588 Treat host permissions as optional in mv3, r=robwu
Comment 3•3 months ago
|
||
Backed out for causing mochitest failures on test_ext_scripting_executeScript.html.
[task 2022-03-12T16:27:56.643Z] 16:27:56 INFO - TEST-PASS | toolkit/components/extensions/test/mochitest/test_ext_scripting_executeScript.html | expected 1 tab - Expected: 1, Actual: 1
[task 2022-03-12T16:27:56.643Z] 16:27:56 INFO - Buffered messages finished
[task 2022-03-12T16:27:56.644Z] 16:27:56 WARNING - TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_scripting_executeScript.html | Test timed out. -
[task 2022-03-12T16:27:56.644Z] 16:27:56 WARNING - TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_scripting_executeScript.html | Extension left running at test shutdown
[task 2022-03-12T16:27:56.644Z] 16:27:56 INFO - SimpleTest.ok@SimpleTest/SimpleTest.js:417:16
[task 2022-03-12T16:27:56.644Z] 16:27:56 INFO - ExtensionTestUtils.loadExtension/<@SimpleTest/ExtensionTestUtils.js:129:18
[task 2022-03-12T16:27:56.644Z] 16:27:56 INFO - executeCleanupFunction@SimpleTest/SimpleTest.js:1487:13
[task 2022-03-12T16:27:56.645Z] 16:27:56 INFO - SimpleTest.finish@SimpleTest/SimpleTest.js:1501:3
[task 2022-03-12T16:27:56.645Z] 16:27:56 INFO - killTest@SimpleTest/TestRunner.js:194:22
[task 2022-03-12T16:27:56.645Z] 16:27:56 INFO - async*delayedKillTest@SimpleTest/TestRunner.js:231:17
[task 2022-03-12T16:27:56.645Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:229:17
[task 2022-03-12T16:27:56.645Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.645Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.646Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.646Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.646Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.646Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.646Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.646Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.647Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.647Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.647Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.647Z] 16:27:56 INFO - setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:242:15
[task 2022-03-12T16:27:56.647Z] 16:27:56 INFO - TestRunner.runTests/<@SimpleTest/TestRunner.js:521:16
[task 2022-03-12T16:27:56.647Z] 16:27:56 INFO - Async*TestRunner.runTests@SimpleTest/TestRunner.js:508:48
[task 2022-03-12T16:27:56.648Z] 16:27:56 INFO - RunSet.runtests@SimpleTest/setup.js:232:14
[task 2022-03-12T16:27:56.648Z] 16:27:56 INFO - RunSet.runall@SimpleTest/setup.js:211:12
[task 2022-03-12T16:27:56.648Z] 16:27:56 INFO - hookupTests@SimpleTest/setup.js:308:12
[task 2022-03-12T16:27:56.648Z] 16:27:56 INFO - parseTestManifest@http://mochi.test:8888/manifestLibrary.js:51:13
[task 2022-03-12T16:27:56.648Z] 16:27:56 INFO - getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:64:28
[task 2022-03-12T16:27:56.648Z] 16:27:56 INFO - EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:60:3
[task 2022-03-12T16:27:56.649Z] 16:27:56 INFO - hookup@SimpleTest/setup.js:284:20
[task 2022-03-12T16:27:56.649Z] 16:27:56 INFO - EventHandlerNonNull*@http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&logFile=%2Fdata%2Flocal%2Ftmp%2Ftest_root%2Flogs%2Fmochitest.log&fileLevel=INFO&consoleLevel=INFO&hideResultsTable=1&manifestFile=tests.json&dumpOutputDirectory=%2Fdata%2Flocal%2Ftmp%2Ftest_root:11:1
[task 2022-03-12T16:27:56.649Z] 16:27:56 INFO - TEST-OK | toolkit/components/extensions/test/mochitest/test_ext_scripting_executeScript.html | took 318734ms
[task 2022-03-12T16:27:56.649Z] 16:27:56 WARNING - TEST-UNEXPECTED-FAIL | /tests/toolkit/components/extensions/test/mochitest/test_ext_scripting_executeScript.html logged result after SimpleTest.finish(): Extension left running at test shutdown
[task 2022-03-12T16:27:56.649Z] 16:27:56 INFO - TEST-START | toolkit/components/extensions/test/mochitest/test_ext_scripting_insertCSS.html
Flags: needinfo?(tomica)
Pushed by tjovanovic@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/669b00e88416 Treat host permissions as optional in mv3, r=robwu
Assignee | ||
Updated•3 months ago
|
Flags: needinfo?(tomica)
Comment 5•3 months ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 3 months ago
status-firefox100:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
Updated•2 months ago
|
Assignee | ||
Updated•3 days ago
|
Summary: Treat host_permissions as optional in MV3 manifest → Treat host permissions as optional in MV3 manifest
You need to log in
before you can comment on or make changes to this bug.
Description
•