Bug 15233 – layout tests try to load file:///tmp/ files on windows

• Component: Tools / Tests
• Status: RESOLVED INVALID

[Tony Chang]

There are four layout tests that try to load local files using the URL file:///tmp/LayoutTests/.  These tests are for checking the ability of local html files to load local resources.  On Windows, the file:///tmp/LayoutTests/ path is not valid because it is a cygwin path and Safari doesn't know about cygwin paths.

We should be able to change the URLs to be relative to test local file loads and work with Windows file:/// paths.

Patch coming up...

[Tony Chang]

Created attachment 16314 [details]
use relative file:/// URLs

I wasn't able to get DumpRenderTree working on Windows, but the tests pass on DRT Mac and when run manually in Safari on Windows.

[Adam Roben (:aroben)]

Comment on attachment 16314 [details]
use relative file:/// URLs

This patch won't fix the versions of these tests that reside in http/tests/security. I think we want a solution that will work for both.

[Tony Chang]

(In reply to comment #2)
> This patch won't fix the versions of these tests that reside in
> http/tests/security. I think we want a solution that will work for both.

Oh, I didn't see those tests.  Maybe we can replace the http tests to be php files that use cygpath to get the windows pathname?

[Tony Chang]

This was fixed in r28690 by adding layoutTestController.pathToLocalResource.