Closed Bug 1189871 Opened 4 years ago Closed 4 years ago
Add event to add/remove permissions for Graphene
There should be an event in b2g/chrome/content/shell.js which wraps setting and removing permissions. It would for example call: Services.perms.add(slashdotURI, 'trackingprotection', Services.perms.ALLOW_ACTION);
Summary: Add API to add/remove permissions → Add event to add/remove permissions for Graphene
Attachment #8642097 - Flags: review?(fabrice) → review+
url: https://hg.mozilla.org/integration/mozilla-inbound/rev/4595fcf7f37090d7799ad6478651677747dbe5c0 changeset: 4595fcf7f37090d7799ad6478651677747dbe5c0 user: Brian R. Bondy <email@example.com> date: Mon Aug 10 17:00:01 2015 -0400 description: Bug 1189871 - Add event to add/remove permissions for Graphene. r=fabrice
Any reason why you need to create these events than navigator.mozPermissionSettings other than both you and your reviewer does not know it exists? https://developer.mozilla.org/en-US/docs/Web/API/PermissionSettings
If it does the same and was available when this landed on larch, then no.
Could you file a bug to get these event listeners removed, once you remove the dependency? It would be better if we leverage existing and testable code path. I don't really know where this is being used... Thanks!
Done, thanks for the suggestion.
You need to log in before you can comment on or make changes to this bug.