WebKit Bugzilla
New
Browse
Log In
×
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
Bug 15233
layout tests try to load file:///tmp/ files on windows
https%3A%2F%2Fbugs.webkit.org%2Fshow_bug.cgi?id=15233
Summary
layout tests try to load file:///tmp/ files on windows
Tony Chang
Reported
2007-09-18 11:22:25 PDT
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...
Attachments
use relative file:/// URLs
(4.04 KB, patch)
2007-09-18 11:35 PDT
,
Tony Chang
aroben
: review-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Note
You need to
log in
before you can comment on or make changes to this bug.
Tony Chang
Comment 1
2007-09-18 11:35:26 PDT
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 2
2007-09-18 12:47:40 PDT
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
Comment 3
2007-09-18 13:10:34 PDT
(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
Comment 4
2008-01-07 16:48:05 PST
This was fixed in
r28690
by adding layoutTestController.pathToLocalResource.
Top of Page
Format For Printing
XML
Clone This Bug