1653729 - Generalize do_QueryActor APIs with support for JSProcessActor

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, enhancement, P3)

Comment 1

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

Attachment: Bug 1653729 - Part 1: Generalize do_QueryActor API,

This changes the set of types which can be passed as the second argument to
do_QueryActor to ones which can be unambiguously converted to a JSActorManager,
and combines nsCOMPtr_helper implementations.

Comment 2

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

Attachment: Bug 1653729 - Part 2: Add methods for do_QueryActor from nsIDOMProcess*,

Before this change, nsIDOMProcess{Parent,Child} could not directly be used in
do_QueryActor, as they don't directly inherit from JSActorManager.

The std::enable_if_t trickery is required in order to avoid overload issues when
passing types like ContentChild which inherit from both JSActorManager and
nsIDOMProcess{Parent,Child}.

Comment 3

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

Attachment: Bug 1653729 - Part 3: Add nsIDOMProcessChild::GetSingleton helper,

This should make it easier to get an XPCOM interface from a JSProcessActorChild
in the current process, when combined with the do_QueryActor overloads from p2.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9db5db7c2b0e
Part 1: Generalize do_QueryActor API, r=kmag
https://hg.mozilla.org/integration/autoland/rev/6169f483a5ca
Part 2: Add methods for do_QueryActor from nsIDOMProcess*, r=kmag
https://hg.mozilla.org/integration/autoland/rev/ef6b1b08c555
Part 3: Add nsIDOMProcessChild::GetSingleton helper, r=kmag

Comment 5

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/9db5db7c2b0e
https://hg.mozilla.org/mozilla-central/rev/6169f483a5ca
https://hg.mozilla.org/mozilla-central/rev/ef6b1b08c555