1609128 - Add polymorphic type support to protocol.js

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement)

Description

[Alexandre Poirot [:ochameau]]

For now, you can only return a unique type of actor from an actor method, or pass a unique type of front to a front method argument.
But in many case, we do return more than one possible type of actor.
For example, the target actors could be a BrowsingContextTargetActor inherited actor, or a ContentProcessTargetActor.
In order to support that, we have to avoid using protocol.js types and use json as a type. Then we need to manually convert the actor to its form, via actor.form() on the actor side. And on the client side, we have to manually instantiate the front from the form.

It would be nice to add some support for polymorphism, so that we can return more than just one type of actor and have protocol.js to automatically pass the form on the wire, and instantiate the right Front class on the client side.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1609128 - Implement polymorphism in protocol.js

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1609128 - Refactor test_protocol_children.js to use Task/async/await.

Comment 3

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1609128 - Move all protocol.js tests next to its implementation.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/92d116a3c59d
Refactor test_protocol_children.js to use Task/async/await. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/1e72656e038e
Move all protocol.js tests next to its implementation. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/373c0e15c266
Implement polymorphism in protocol.js r=jdescottes

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/92d116a3c59d
https://hg.mozilla.org/mozilla-central/rev/1e72656e038e
https://hg.mozilla.org/mozilla-central/rev/373c0e15c266