Bug 55089 – [chromium] Move media-file.js and video-test.js back to media/ for UILayoutTests

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Victoria Kirst]

[chromium] Move media-file.js and video-test.js back to media/ for UILayoutTests

[Victoria Kirst]

Created attachment 83554 [details]
Patch

[Victoria Kirst]

Created attachment 83562 [details]
Patch

[Victoria Kirst]

Moving media-file.js and video-test.js to http/tests/media/ caused Chromium's MediaUILayoutTests to fail (http://crbug.com/73287), so I moved them back to the media/ folder.

In order for the http tests to access these files, I added an alias to the media/ folder in the scripts that configure the httpd server.

[WebKit Commit Bot]

Comment on attachment 83562 [details]
Patch

Rejecting attachment 83562 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sl', 'bu..." exit_code: 2

Last 500 characters of output:

java/lc3/JSUndefined ..........
java/lc3/JavaArray ....
java/lc3/JavaClass ....
java/lc3/JavaObject .....................................
java/lc3/StringMethods .
java/lc3/forin .
java/lc3/instanceof .
loader ..
mathml .......
mathml/presentation ..................
media .
media/adopt-node-crash.html -> failed

Exiting early after 1 failures. 13006 tests run.
295.39s total testing time

13005 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
9 test cases (<1%) had stderr output

Full output: http://queues.webkit.org/results/7984514

[Mihai Parparita]

I forgot that patches that move files can't be landed through the commit queue (see bug 49154). Will land this by hand.

[Mihai Parparita]

Committed r79598: <http://trac.webkit.org/changeset/79598>

[Victoria Kirst]

Thanks for landing, Mihai!

[James Robinson]

Reverted r79598 for reason:

Appears to be causing trouble with httpd on windows

Committed r79623: <http://trac.webkit.org/changeset/79623>

[Mihai Parparita]

(In reply to comment #8)
> Reverted r79598 for reason:
> 
> Appears to be causing trouble with httpd on windows
> 
> Committed r79623: <http://trac.webkit.org/changeset/79623>

Based on http://www.cyberciti.biz/tips/configure-lighttpd-alias-mod_alias.html it looks like subsequent alias.url lines need to use += in the lighthttpd config.

[Mihai Parparita]

(In reply to comment #9)
> Based on http://www.cyberciti.biz/tips/configure-lighttpd-alias-mod_alias.html it looks like subsequent alias.url lines need to use += in the lighthttpd config.

The logs show this too (from http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Win/builds/2045/steps/webkit_tests/logs/stdio)

Duplicate config variable in conditional 0 global: alias.url
2011-02-24 14:19:12: (configfile.c.855) source: e:\b\build\slave\Webkit_Win\build\src\webkit\Release\..\../../layout-test-results\lighttpd.conf line: 105 pos: 1 parser failed somehow near here: (EOL)

[Mihai Parparita]

Committed r79630: <http://trac.webkit.org/changeset/79630>

[WebKit Review Bot]

http://trac.webkit.org/changeset/79598 might have broken GTK Linux 32-bit Release

[WebKit Review Bot]

http://trac.webkit.org/changeset/79623 might have broken GTK Linux 64-bit Debug