[webkit-reviews] review denied: [Bug 27492] Web Sockets Test Infrastructure Part 3/3: Example test : [Attachment 34056] Example Web Socket Tests, same-origin and cross-origin.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Aug 7 13:01:44 PDT 2009
Eric Seidel <eric at webkit.org> has denied Yuzo Fujishima <yuzo at google.com>'s
request for review:
Bug 27492: Web Sockets Test Infrastructure Part 3/3: Example test
https://bugs.webkit.org/show_bug.cgi?id=27492
Attachment 34056: Example Web Socket Tests, same-origin and cross-origin.
https://bugs.webkit.org/attachment.cgi?id=34056&action=review
------- Additional Comments from Eric Seidel <eric at webkit.org>
Great that these are js tests! Bad that you had to copy the files. We really
should fix bug 25915 instead.
ChangeLog is duplicated.
13 debug('<span class="pass">TEST COMPLETE</span>');
is instead a testPassed() call.
Style:
if (window.layoutTestController) {
4 window.layoutTestController.waitUntilDone();
5 }
description() is the right call here:
debug("NOOP Web Socket test");
r- for various nits above.
More information about the webkit-reviews
mailing list