WebSocket layout test cleanup

WebSocket layout test cleanup

- Rename "Web Socket" in layout tests
- Rephrase comments
- Make comments fit 80 col
- Fix comment referring old spec hixie-60

BUG=none

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

[yhirano, 2013-10-30 10:42:53]

I think send-throw and close-before-open should be fixed as well.

[tyoshino (SeeGerritForStatus), 2013-10-30 10:54:12]

On 2013/10/30 10:42:53, yhirano wrote:
> I think send-throw and close-before-open should be fixed as well.

done

[yhirano, 2013-10-30 11:35:07]

lgtm

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
File
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe-reference-in-parent.html
(right):

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe-reference-in-parent.html:13:
// send "socket1" to server, so that "socket1" will be broadcasted to
"Send" instead of "send"?

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
File
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe.html
(right):

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe.html:12:
// send "socket1" to server, so that "socket1" will be broadcasted to
"Send" instead of "send"?

[tyoshino (SeeGerritForStatus), 2013-10-30 11:56:19]

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
File
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe-reference-in-parent.html
(right):

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe-reference-in-parent.html:13:
// send "socket1" to server, so that "socket1" will be broadcasted to
On 2013/10/30 11:35:07, yhirano wrote:
> "Send" instead of "send"?

Done.

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
File
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe.html
(right):

https://codereview.chromium.org/52103004/diff/50001/LayoutTests/http/tests/we...
LayoutTests/http/tests/websocket/tests/hybi/resources/close-on-unload-iframe.html:12:
// send "socket1" to server, so that "socket1" will be broadcasted to
On 2013/10/30 11:35:07, yhirano wrote:
> "Send" instead of "send"?

Done.

[commit-bot: I haz the power, 2013-10-30 11:56:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tyoshino@chromium.org/52103004/130001

[commit-bot: I haz the power, 2013-10-30 11:57:08]

Failed to apply patch for
LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file
LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt
  Hunk #1 FAILED at 1.
  1 out of 1 hunk FAILED -- saving rejects to file
LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt.rej

Patch:      
LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt
Index:
LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt
diff --git
a/LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt
b/LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt
index
ce415b28beeb8357946d20f5f3c37360af921c7c..d9381a22183041ebccf3e1acf8429527d6428efd
100644
--- a/LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt
+++ b/LayoutTests/http/tests/websocket/tests/hybi/close-before-open-expected.txt
@@ -1,6 +1,6 @@
 CONSOLE WARNING: WebSocket connection to
'ws://127.0.0.1:8880/websocket/tests/hybi/echo' failed: WebSocket is closed
before the connection is established.
 CONSOLE WARNING: WebSocket connection to
'ws://127.0.0.1:8880/websocket/tests/hybi/close-code-and-reason' failed:
WebSocket is closed before the connection is established.
-Test if Web Socket fires close event when WebSocket is opened and closed fore
open event is received.
+Test if WebSocket fires close event when WebSocket is opened and closed fore
open event is received.
 
 On success, you will see a series of "PASS" messages, followed by "TEST
COMPLETE".

[commit-bot: I haz the power, 2013-10-30 12:32:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tyoshino@chromium.org/52103004/130002

[commit-bot: I haz the power, 2013-10-30 13:39:06]

Change committed as 160931