[Webkit-unassigned] [Bug 125339] Enable running layout tests needing http server outside LayoutTests/http/tests

[bugzilla-daemon at webkit.org]

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





--- Comment #12 from youenn fablet <youennf at gmail.com>  2013-12-17 07:30:18 PST ---
(In reply to comment #9)
> I haven't done a detailed review of this yet. My initial reaction is that this adds more complexity to the setup than I'd really like; my initial leaning is that we should just serve *all* of LayoutTests/imported/w3c from http and call it good enough.

Ideally, it would be great to have one single straightforward mapping rule between test names and urls.
As a newcomer though, I find it as convenient to grasp the mapping rules from a config file than from python source files(personal taste probably).
Also, that would allow putting in that same config file the existing aliases (media-resources and js-test-resources).

> However, I think WebKit still only runs all of the http tests in a single thread, and even Blink only runs them in 25% of threads , so if we start running a lot of imported tests, this might be a bottle neck. It would be nice if there was some way other than manually tracking which directories did and did not actually require a web server, but I'm not sure if the w3c is going to be super excited to track that, as their basic model assumes tests are run off of a web server.
f> 
> Perhaps the better thing to do is to focus on being able to run more tests in parallel off a web server without flakiness, I'm not sure.
> 
> At any rate, I'd like to do a more detailed review of this patch, but I don't want to block WebKit on this; if someone else (e.g., rniwa) wants to review/R+ it (eventually) and not wait on me, that's fine.
> 
> (I'm currently working again on the import process in Blink; where possible, I'd like to keep things as close to WebKit as possible, which is why I'm waffling on this at all).

If possible, I would like to go on with this patch to start adding some w3c XHR tests.
When you finalize a better solution in Blink, I will be happy to help in porting it within webkit.

> Other things I'd note are that I think the map of which directories need or don't need the server should probably live at the top level under LayoutTests/ rather than in http/tests/conf . 

I will update the patch accordingly

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