917254 - Only enable MessageEvent.ports when MessagePort is enabled

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[Andrea Marchesini [:baku]]

Attachment: patch

Comment 2

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

Should we get this to Aurora too?

Comment 3

[(no longer active)]

(In reply to comment #2)
> Should we get this to Aurora too?

Yes!

Comment 4

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7912be7f71db

Comment 5

[Andrea Marchesini [:baku]]

Comment on attachment 805975 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 848294
User impact if declined: MessageEvent.port can be exposed but it doesn't work.
Risk to taking this patch (and alternatives if risky): MessagePort is landed but it's disabled by pref. This MessageEvent.port is part of this new MessagePort feature but it's not disabled.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7912be7f71db

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/9c001497df0f

Comment 8

[Tracy Walker [:tracy]]

Is there anything QA can do here to verify this?