1313683 - Remove PContent::Msg_ReadPermissions sync message

Status: RESOLVED DUPLICATE of bug 1337056

(Core :: Widget, defect, P4)

Description

[Mike Conley (:mconley) (:⚙️)]

When nsPermissionManager::Init fires in the content process, it sends a sync message "ReadPermissions" to the parent:

http://searchfox.org/mozilla-central/rev/445097654c1cc1098ee0171a29c439afe363a588/extensions/cookie/nsPermissionManager.cpp#795

It looks like this only occurs once the permission manager starts up, but it also looks like there's an API for refreshing permissions (nsIPermissionManager.refreshPermission). This was added in bug 1129315 for b2g, but I don't think it's called anymore. We might want to remove that.

We might be able to piggyback on the work going on in bug 1303096 to send permissions down along with the look-and-feel cache and preferences.

Comment 1

[(no longer active)]

(In reply to Mike Conley (:mconley) from comment #0)
> When nsPermissionManager::Init fires in the content process, it sends a sync
> message "ReadPermissions" to the parent:
> 
> http://searchfox.org/mozilla-central/rev/
> 445097654c1cc1098ee0171a29c439afe363a588/extensions/cookie/
> nsPermissionManager.cpp#795
> 
> It looks like this only occurs once the permission manager starts up, but it
> also looks like there's an API for refreshing permissions
> (nsIPermissionManager.refreshPermission). This was added in bug 1129315 for
> b2g, but I don't think it's called anymore. We might want to remove that.

Bug 1337056's fix is removing this sync message.  There's some evidence there so I'll dupe this way.

> We might be able to piggyback on the work going on in bug 1303096 to send
> permissions down along with the look-and-feel cache and preferences.

I actually have a better plan!  I'll post it to that bug.