1791747 - AutoSyncLoopHolder::GetEventTarget should return nsISerialEventTarget (instead of nsIEventTarget)

Status: RESOLVED FIXED

(Core :: DOM: Workers, task, P2)

Description

[Jan Varga [:janv]]

This makes it easier to use with mozPromise which expects nsISerialEventTarget, otherwise a QueryInterface is needed which is kind of ugly.

Comment 1

[Jan Varga [:janv]]

Attachment: Bug 1791747 - AutoSyncLoopHolder::GetEventTarget should return nsISerialEventTarget (instead of nsIEventTarget); r=#dom-workers

Depends on D157803

Comment 2

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

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:janv, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Up to janv to land.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/caf612a41819
AutoSyncLoopHolder::GetEventTarget should return nsISerialEventTarget (instead of nsIEventTarget); r=dom-worker-reviewers,smaug

Comment 5

[Serban Stanca [:SerbanS]]

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

Comment 6

[Jan Varga [:janv]]

Landed.