1537977 - Add a ManagedEndpoint<PFooSide> type to allow for more complex actor construction

Status: RESOLVED FIXED

(Core :: IPC, enhancement, P2)

Description

[Nika Layzell [:nika] (ni? for response)]

This is specifically motivated by the desire to be able to allocate certain actors synchronously with their managers being connected. It is handled by passing around a type, like Endpoint<T>, which is then used to bind the actors on either side.

No messages may be sent before the actors are constructed, or routing errors may occur.

Comment 1

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1537977 - Add a ManagedEndpoint IPDL type for manually constructing actors,

Comment 2

[Andrew McCreight [:mccr8]]

How is this different from Endpoint? It has been a while since I looked at them.

Comment 3

[Nika Layzell [:nika] (ni? for response)]

Endpoint is used to construct toplevel actors by establishing an underlying transport pipe which can be shared between the source & destination processes. These can then be linked to open a toplevel actor, and is how actors such as PBackground are created.

In contrast, ManagedEndpoint is intended to be used for constructing managed actors, which at the transport layer share the same pipe as the root actor, and route their messages to the correct destination based on the routing_id attached to the message.

Comment 4

[Andrew McCreight [:mccr8]]

(In reply to :Nika Layzell from comment #3)

In contrast, ManagedEndpoint is intended to be used for constructing managed actors, which at the transport layer share the same pipe as the root actor, and route their messages to the correct destination based on the routing_id attached to the message.

Thanks for the explanation! I've done a bit of work with Endpoints, but it was all for various GMP stuff, and I hadn't thought about how they were all top level actors.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8cde0ca01548
Add a ManagedEndpoint IPDL type for manually constructing actors, r=mccr8

Comment 6

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/8cde0ca01548