[Webkit-unassigned] [Bug 27492] Web Sockets Test Infrastructure Part 3/3: Example test

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27492


David Levin <levin at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #41998|review?                     |review-
               Flag|                            |




--- Comment #11 from David Levin <levin at chromium.org>  2009-11-02 11:45:39 PDT ---
(From update of attachment 41998)
Just some minor issues to address and this should be fine to go in.

> diff --git a/LayoutTests/websocket/tests/resources/simple.js b/LayoutTests/websocket/tests/resources/simple.js

I think this is in the wrong place after Hamaji's changes.


> diff --git a/LayoutTests/websocket/tests/simple_wsh.py b/LayoutTests/websocket/tests/simple_wsh.py
> new file mode 100755
> index 0000000..5dc71bd
> --- /dev/null
> +++ b/LayoutTests/websocket/tests/simple_wsh.py
> @@ -0,0 +1,42 @@
> +# Copyright 2009, Google Inc.
> +# All rights reserved.

Please use the standard copyright style for WebKit.

Ideally the copyright would be of this format:
   Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
   Copyright (C) 2009 Somebody <email>


> +def web_socket_do_extra_handshake(request):
> +    pass  # Always accept.

In keeping with out Webkit code, please use one space before end of line
comments.


> +# vi:sts=4 sw=4 et

No other WebKit sources have this vi formating info. Please remove it.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.