Bug 31390 - Implement run-webkit-websocketserver
Summary: Implement run-webkit-websocketserver
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2009-11-11 18:07 PST by Fumitoshi Ukai
Modified: 2009-12-07 10:35 PST (History)
9 users (show)

See Also:

Add to run-webkit-tests an option to run Web Socket server (2.36 KB, patch)
2009-11-11 23:23 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff
Add run-webkit-websocketserver (4.96 KB, patch)
2009-11-12 23:06 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff
Add run-webkit-websocketserver (5.34 KB, patch)
2009-11-15 22:24 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff
Add run-webkit-websocketserver (4.61 KB, patch)
2009-12-03 21:23 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff
Add run-webkit-websocketserver (4.11 KB, patch)
2009-12-06 21:46 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fumitoshi Ukai 2009-11-11 18:07:17 PST
run-webkit-tests launches websocket server if we tries to test LayoutTests/websocket/tests/

We have a run-webkit-httpd script for debugging HTTP tests in Safari, maybe we need something similar for WebSocket.
Comment 1 Yuzo Fujishima 2009-11-11 23:23:00 PST
Created attachment 43033 [details]
Add to run-webkit-tests an option to run Web Socket server
Comment 2 Yuzo Fujishima 2009-11-11 23:29:52 PST
Hi, reviewers,

Can you review this patch?

This is a Web Socket server counterpart for run-webkit-httpd script.

Rather than writing a separate script, I chose to add an option to run-webkit-tests.
This way it is easy to run the server in the same environment as that for LayoutTests.

Comment 3 David Levin 2009-11-11 23:37:31 PST
This seems really odd to me. Just like there is a "run-webkit-httpd" as you noted, it seems like there should be a separate script for this.
Comment 4 Yuzo Fujishima 2009-11-11 23:48:32 PST
Hi, thank you for the comment.

It may seem odd, but I think it is better than repeating the same/similar
pieces of code in run-webkit-tests and run-webkit-websocket-server or so.

(Because the purpose is testing, although manual, using run-webkit-tests
for this purpose should not be that odd.)

I admit I have very limited knowledge about Perl. If it is easy to "import" 
run-webkit-tests, maybe I should import it to a new script.

Comment 5 Yuzo Fujishima 2009-11-12 23:06:34 PST
Created attachment 43137 [details]
Add run-webkit-websocketserver
Comment 6 Yuzo Fujishima 2009-11-12 23:11:07 PST
OK, I've created another patch, which adds a dedicated script,

If you prefer this dedicated script approach to a new option approach,
that's fine with me.

Can you review the patch(es) and submit whichever you like, assuming
it looks good?

Comment 7 Alexey Proskuryakov 2009-11-13 17:51:32 PST
Ideally, we should share code between run-webkit-tests and run-webkit-websocketserver. We should do this for httpd, as well.

Given that the test server seems to not work on Windows or Tiger, maybe we should investigate those failures first, and refactor launching code once we know what goes wrong? I'm going to see what happens on Tiger.
Comment 8 Yuzo Fujishima 2009-11-15 22:24:33 PST
Created attachment 43266 [details]
Add run-webkit-websocketserver
Comment 9 Yuzo Fujishima 2009-11-15 22:25:18 PST
Disabled wss because currently some platform doesn't support pyOpenSSL.

Comment 10 Eric Seidel (no email) 2009-11-18 15:48:39 PST
FYI, Ojan just re-wrote run-webkit-httpd in Python yesterday (for chromium's own needs -- chromium currently uses lighthttp to run the layout tests and we'd like to move back to apache), an I plan to upstream chromium's python run_webkit_tests.py soon.  Not for the purposes of taking over the world with python (although that's a nice fringe benefit), more so that we can run all of Chromium's WebKit infrastructure right out of webkit.org.  Once I'm successful with that effort, I'd like to roll all of WebKit's extra run-webkit-tests goodness into Chromium's version and combine them in a single replacement for both.

Just mentioning this, as there will be code sharing chances with those scripts as well, even if we don't get to with the perl version attached here.
Comment 11 Adam Barth 2009-11-30 12:25:07 PST
Attachment 43266 [details] passed the style-queue
Comment 12 Yuzo Fujishima 2009-12-03 21:23:46 PST
Created attachment 44289 [details]
Add run-webkit-websocketserver
Comment 13 Yuzo Fujishima 2009-12-03 21:25:56 PST
Updated the patch to log to a file.

Can anyone review and commit this?

(If you don't need this kind of script any more, please close the bug. :) )

Comment 14 WebKit Review Bot 2009-12-03 21:28:40 PST
style-queue ran check-webkit-style on attachment 44289 [details] without any errors.
Comment 15 Alexey Proskuryakov 2009-12-04 12:41:07 PST
Comment on attachment 44289 [details]
Add run-webkit-websocketserver

Thanks for updating the patch!

> +# Note: This script must be run from WebKit directory.

I don't think that there is a good reason for this limitation. Why not use webkitdirs.pm to make it work from any directory, like all our scripts do?

> +    # wss is disabled until all platforms support pyOpenSSL.
> +    # kill 15, $webSocketSecureServerPID;

This needs to be removed prior to landing.

> +    $isWebSocketServerOpen = 0;

Tracking the state seems superfluous - there is no way for openWebSocketServerIfNeeded to be called twice in this script, and no way for it to be called after closing.

r=me, but please fix the script to work from any directory. And this patch cannot be landed by commit-queue, because commented out code needs to be removed.
Comment 16 Yuzo Fujishima 2009-12-06 21:46:27 PST
Created attachment 44383 [details]
Add run-webkit-websocketserver
Comment 17 Yuzo Fujishima 2009-12-06 21:47:19 PST

Addressed the comments. Can you take another look?

Comment 18 WebKit Review Bot 2009-12-06 21:50:03 PST
style-queue ran check-webkit-style on attachment 44383 [details] without any errors.
Comment 19 Alexey Proskuryakov 2009-12-07 10:21:54 PST
Comment on attachment 44383 [details]
Add run-webkit-websocketserver

Comment 20 WebKit Commit Bot 2009-12-07 10:34:57 PST
Comment on attachment 44383 [details]
Add run-webkit-websocketserver

Clearing flags on attachment: 44383

Committed r51779: <http://trac.webkit.org/changeset/51779>
Comment 21 WebKit Commit Bot 2009-12-07 10:35:04 PST
All reviewed patches have been landed.  Closing bug.