Have postMessage()s throw TypeError on failed arity checks.

Have postMessage()s throw TypeError on failed arity checks.

R=haraken,mkwst@chromium.org
BUG=339441
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192337

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192483

[sof, 2015-03-23 06:33:15]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, jsbell@chromium.org, mkwst@chromium.org

[sof, 2015-03-23 06:33:16]

please take a look.

It might be worth auditing other custom methods for same?

[Mike West, 2015-03-23 06:40:52]

LGTM. Thanks for cleaning this up!

[haraken, 2015-03-23 07:28:20]

LGTM

[sof, 2015-03-23 08:19:42]

thanks; should there be tests (or content) that rely on worker.postMessage()
posting 'undefined', we'll just have to work through those issues.

[sof, 2015-03-23 08:19:46]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-03-23 08:19:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1025203002/20001

[commit-bot: I haz the power, 2015-03-23 08:24:45]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=192337

[dglazkov, 2015-03-23 14:59:40]

On 2015/03/23 at 08:24:45, commit-bot wrote:
> Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=192337

Is this commit causing this problem here?
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...

[dglazkov, 2015-03-23 15:05:18]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1033443002/ by dglazkov@chromium.org.

The reason for reverting is: Broke
org.chromium.android_webview.test.PostMessageTest#testPostUnsupportedWebMessageToApp

http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r....

[sof, 2015-03-24 15:19:10]

With https://codereview.chromium.org/1003603004/ addressed, this can be
attempted relanded, but will wait for
https://codereview.chromium.org/1027333003/ also.

(https://code.google.com/p/chromium/codesearch#search/&q=%5C.postMessage%5C(%5...
shows up no other non-throwing uses of postMessage().)

[sof, 2015-03-24 19:08:16]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-03-24 19:08:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1025203002/20001

[commit-bot: I haz the power, 2015-03-24 21:31:57]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=192483