Closed
Bug 1632091
Opened 4 years ago
Closed 11 months ago
Add test to verify that permissions in NO_PROMPT_PERMISSIONS does not have a message string
Categories
(WebExtensions :: General, task, P3)
WebExtensions
General
Tracking
(firefox115 fixed)
RESOLVED
FIXED
115 Branch
Tracking | Status | |
---|---|---|
firefox115 | --- | fixed |
People
(Reporter: robwu, Assigned: eemeli)
References
Details
Attachments
(1 file)
bug 1622917 introduced an explicit list of permissions for which we shouldn't prompt.
bug 1631137 partially removed the check in one place and started to rely on the non-existence of a permission warning message in order to determine whether a permission should be prompted for.
To make sure that these two things are in sync, let's add a test for it. It probably suffices to add one extra line at https://searchfox.org/mozilla-central/rev/3446310d6cc5c85cde16a82eccf560e9b71a3d44/toolkit/components/extensions/test/xpcshell/test_ext_permissions.js#604
Updated•4 years ago
|
Assignee: nobody → mixedpuppy
Severity: -- → minor
Priority: -- → P2
Updated•4 years ago
|
Assignee: mixedpuppy → nobody
Severity: minor → S4
Priority: P2 → P3
Assignee | ||
Comment 1•1 year ago
|
||
Depends on D158663
Updated•1 year ago
|
Assignee: nobody → earo
Status: NEW → ASSIGNED
Pushed by earo@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cf5d816c4bf0 Test extension permission description data consistency. r=robwu
Comment 3•11 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
status-firefox115:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•