Closed
Bug 1459941
Opened 7 years ago
Closed 5 years ago
Let geolocation be an optional permission
Categories
(WebExtensions :: General, enhancement, P3)
WebExtensions
General
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1618398
People
(Reporter: aswan, Assigned: robwu)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
I thought we talked about doing this a while ago but can't find a bug for it.
It recently came up again in bug 1459229.
|
||
Updated•7 years ago
|
Priority: -- → P3
|
||
Updated•6 years ago
|
Product: Toolkit → WebExtensions
|
Assignee | |
Comment 1•5 years ago
|
||
geolocation is declared as an OptionalPermission in the manifest schema, but currently the permission is only checked when the extension starts up. This is flawed, we need to listen for the "add-permission" and "remove-permission" events and update the permission in the permission manager accordingly.
Marking as dependency of bug 1589693, because the implementation of geolocation and notifications are expected to be similar, and if we implement the behavior for "notifications" anyway, then we may as well fix the behavior of geolocation.
Depends on: 1589693
|
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → rob
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
|
Assignee | |
Updated•5 years ago
|
|
||
Comment 3•5 years ago
|
||
geolocation was already optional and the permission updating was addressed in bug 1618398
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•