1611426 - Cleanup nsContentBlocker permissions

Status: RESOLVED FIXED

(Core :: Permission Manager, task, P3)

Description

[Emma Zühlcke [:emz]]

Now that nsContentBlocker has been removed in Bug 1357107, we should clean up the leftover permissions from the profiles.

Permissions used by nsContentBlocker

It would probably make sense to do this in FF 75 earliest, after the nsContentBlocker removal hit release.

Comment 1

[Andrea Marchesini [:baku]]

Paul, any news on this bug?

Comment 2

[Emma Zühlcke [:emz]]

I'm currently busy with other tasks but this is definitely something I'd like to get back to. However, I don't want to block anyone else from picking this up!

Comment 3

[Emma Zühlcke [:emz]]

To clean up these old permissions you can add a migration in BrowserGlue.sys.mjs, see https://searchfox.org/mozilla-central/rev/2a867dd1ab015c3ef24b774a57709fb3b3dc4961/browser/components/BrowserGlue.sys.mjs#3802. The nsIPermissionManager interface is accessible via Services.perms and allows you to clear the permission which match the type list here: https://searchfox.org/mozilla-central/rev/ba4fab1cc2f1c9c4e07cdb71542b8d441707c577/extensions/permissions/nsPermissionManager.cpp#107-118

Comment 4

[William Wen (:wwen)]

Attachment: Bug 1611426 - Add migration to delete permissions no longer used by nsContentBlocker. r=pbz

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/98d624d1597e
Add migration to delete permissions no longer used by nsContentBlocker. r=pbz,firefox-desktop-core-reviewers ,Gijs

Comment 6

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/98d624d1597e