1026419 - Implement PContentContent to manage communications between contents

Status: RESOLVED WONTFIX

(Core :: DOM: Content Processes, defect, P5)

Description

[Kan-Ru Chen [:kanru] (UTC+9)]

Create a new protocol PContentContent to manage communications between contents like SyncMessage, AsyncMessage, RpcMessage, PBrowser creation, CPOW creations, PBlob creations, etc.

Currently for nested oop we use two top level protocols PContent and PContentBridge to connect chrome<->content and content<->content respectively. Then we use nsIContentParent to implement the common bits. However, implementing in this way means if we modify the protocols we need to change the function signature of Content{Parent,Child}, ContentBridge{Parent,Child}, nsIContent{Parent,Child}. This is very error prone.

The proposed new protocol structure is:

PContent manages PContentContent
PContentBridge manages PContentContent
PContentContent managed by PContent or PContentBridge
PContentContent manages PJavaScript, PBrowser, PBlob

We might want to change the protocol names to be saner. Something like

PContent -> PProcess
PContentBridge -> PProcessBridge
PContentContent -> PContent?

but we don't have to worry about this in this bug.

Comment 1

[David Zbarsky (:dzbarsky)]

I originally wanted to rename PContent to PProcess, but I believe Chris Jones had some argument against it.  It may be a moot point now.

Comment 2

[Kan-Ru Chen [:kanru] (UTC+9)]

Attachment: WIP: Add PContentContent.ipdl and impl

Comment 3

[Kan-Ru Chen [:kanru] (UTC+9)]

Attachment: WIP: Make PBlob be managed by PContentContent

Comment 4

[Kan-Ru Chen [:kanru] (UTC+9)]

Attachment: WIP: Make PJavaScript be managed by PContentContent

Comment 5

[Kan-Ru Chen [:kanru] (UTC+9)]

Attachment: WIP: Move PBrowser to be managed by PContentContent

Comment 6

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.

Comment 7

[Jonathan Watt [:jwatt]]

Ryan, I imagine this can be closed since bug 1500257 landed?

Comment 8

[Ryan Hunt [:rhunt] (on leave until early May)]

Yes this should be closed.