758270 - Electrolysize Permission Manager

Status: RESOLVED WORKSFORME

(Core :: General, defect)

Description

[Jim Straus]

Permission Manager needs to support multiple processes for B2G.  Create multi-process for Permission Manager.  Child processes could subscribe to changes and cache results for quick answers when the decision is already made.

Comment 1

[Jim Straus]

Attachment: WIP PermissionManager.ipdl

Here is an ipdl for the electrolization of PermissionManager.  It isn't clear how the multi-process Gecko is going to work.  I'm not sure how this communication will be set up, as I expect there will be several ipdls needing to communicate to the children content processes.  Also, I don't think the WebAPI versions of testPermission will need to be available to the children content processes (as I don't expect the APIs to live in the children content processes).  I can eliminate them from the interface once it is known.  I did include a notification of changes so that the children versions can cache results for quicker answers.  Until there is more infrastructure in place, I can't assess the security implications.

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Isn't this already working? How could we have shipped XUL-Fennec which was multi-process if the permission manager wasn't working in child processes?

Blocking on figuring this out either way though.

Comment 3

[Josh Matthews [:jdm]]

Yes, nsPermissionManager fully supports multiple content processes.

Comment 4

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Jim: should this be marked as WORKSFORME then? Or did you have any additions in mind?