1729043 - [GMP] Remove intr usage from PGMP.ipdl

Status: RESOLVED FIXED

(Core :: Audio/Video: GMP, task, P3)

Description

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

We should rework this intr usage. See bug 1729036 for motivation.

Comment 1

[Jed Davis [:jld] ⟨⏰|UTC-7⟩ ⟦he/him⟧]

PGMP::StartPlugin is effectively a sync message in the parent-to-child direction; normal sync messages are restricted to be child-to-parent, but nested sync messages can bypass that restriction and can be used here.

Comment 2

[Jed Davis [:jld] ⟨⏰|UTC-7⟩ ⟦he/him⟧]

Attachment: Bug 1729043 - Remove `intr` from the `PGMP` protocol.

PGMP::StartPlugin is intr in order to be a sync message in the
parent-to-child direction; normal sync messages must be child-to-parent.
However, nested sync messages have looser restrictions; this patch
changes the message to do that.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:jld, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ada19d93895f
Remove `intr` from the `PGMP` protocol. r=nika,bryce

Comment 5

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/ada19d93895f