Layout test to verify pong message from the server.

Layout test to verify pong message from the server.

Issue crbug.com/443651 says that Chrome crashes on receiving pong
messages with a payload. This does not appear to be true, but there
is not an existing layout test testing this case, so add one.

BUG=443651
TEST=layout tests

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

[Adam Rice, 2014-12-18 19:53:03]

ricea@chromium.org changed reviewers:
+ tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2015-01-05 05:19:45]

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
File LayoutTests/http/tests/websocket/pong-only.html (right):

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
LayoutTests/http/tests/websocket/pong-only.html:11: var message;
let's declare these variables right above each handler as nothing else uses them

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
LayoutTests/http/tests/websocket/pong-only.html:28: 
remove blank line?

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
File LayoutTests/http/tests/websocket/pong-only_wsh.py (right):

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
LayoutTests/http/tests/websocket/pong-only_wsh.py:1: import cgi
needed?

[tyoshino (SeeGerritForStatus), 2015-01-05 05:24:29]

Let's give a file name that describes what the file does if possible than using
a term "-only" which we need to compare with something else to understand the
meaning.

For example:
receive-pong-with-payload.html
receive-pong-with-payload_wsh.py

[Adam Rice, 2015-01-05 20:24:32]

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
File LayoutTests/http/tests/websocket/pong-only.html (right):

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
LayoutTests/http/tests/websocket/pong-only.html:11: var message;
On 2015/01/05 05:19:45, tyoshino wrote:
> let's declare these variables right above each handler as nothing else uses
them

Done.

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
LayoutTests/http/tests/websocket/pong-only.html:28: 
On 2015/01/05 05:19:45, tyoshino wrote:
> remove blank line?

Done.

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
File LayoutTests/http/tests/websocket/pong-only_wsh.py (right):

https://codereview.chromium.org/788923007/diff/1/LayoutTests/http/tests/webso...
LayoutTests/http/tests/websocket/pong-only_wsh.py:1: import cgi
On 2015/01/05 05:19:45, tyoshino wrote:
> needed?

Removed.

[tyoshino (SeeGerritForStatus), 2015-01-06 03:03:21]

lgtm

[Adam Rice, 2015-01-06 09:11:47]

The CQ bit was checked by ricea@chromium.org

[commit-bot: I haz the power, 2015-01-06 09:13:05]

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

[commit-bot: I haz the power, 2015-01-06 09:16:19]

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