Bug 63141 – Wrong argument order in window.postMessage

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Yury Semikhatsky]

The specification says(http://dev.w3.org/html5/postmsg/#posting-messages) that the order of window.postMessage arguments is (message, targetOrigin, ports) while current WebKit implementation expects optional ports argument to go second i.e. (message, ports, targetOrigin). It was implemented this way in WebKit a couple of years ago(http://trac.webkit.org/changeset/32597/trunk/WebCore/bindings/js/JSDOMWindowCustom.cpp) so I'm not sure what we should do about this now when there may be some uses of that implementation.

[Yury Semikhatsky]

Sorry, messages ports were introduced in r36891: http://trac.webkit.org/changeset/36891/trunk/WebCore/bindings/js/JSDOMWindowCustom.cpp

[Adam Barth]

Ouch.  I wonder if we should change the spec.  Does Firefox match the spec?

[Alexey Proskuryakov]

We certainly matched the spec at the time of adding the code.

[Ian 'Hixie' Hickson]

If you plan to change this, hold off a few days because I'm about to update this API anyway and I wouldn't want to end up changing it around and having you waste your time.

The problem with the order (message, ports, targetOrigin) is that it means you don't know what the second argument is unless you check its type or check the number of arguments. With the order (message, targetOrigin, ports) the optional argument is the last one, not the middle one.

Does anyone else support sending ports yet? If not this might be a moot point and we might be able to change it easily. How much WebKit-specific content is there out there using this?

[Yury Semikhatsky]

(In reply to comment #4)
> 
> Does anyone else support sending ports yet? If not this might be a moot point and we might be able to change it easily. How much WebKit-specific content is there out there using this?

Checked with Firefox 5.0 on Mac. LayoutTests/fast/events/message-port.html fails on attempt to create MessageChannel - there is no such constructor. Firefox doesn't support SharedWorkers either. So I don't see any way even to create a MessagePort in Firefox.

[johnjbarton]

It looks like the WebKit API is
  otherWindow.postMessage(message, ports);
Specifically a third argument is not allowed. V8 at least requires the |ports| to be an Array.

[johnjbarton]

(In reply to comment #6)
> It looks like the WebKit API is
>   otherWindow.postMessage(message, ports);
> Specifically a third argument is not allowed. V8 at least requires the |ports| to be an Array.

At least in Chrome (V8), the third argument is required and is checked as a targetOrigin. So perhaps I am not reading the WebKit source correctly.

[Ian 'Hixie' Hickson]

Looks like WebKit actually supports the second and third arguments in either order. Is that intentional?

[Ian 'Hixie' Hickson]

I've put that in the spec, to hopefully end this confusion about which argument is second and which is third once and for all. Looks like only WebKit implements MessagePort through postMessage() correctly enough for this to matter, and the spec now matches WebKit.

[Ian 'Hixie' Hickson]

Actually nevermind, my testing was bogus.

[Ian 'Hixie' Hickson]

Opera implements the spec as written here. Having the optional argument in the middle (as WebKit has it) seems generally bad, so if we're not going to support either order, I guess we should just go with what's in the spec now.

Please let me know if this causes compat issues.

[Sam Weinig]

Fixed in r209303: <http://trac.webkit.org/changeset/209303>