Bug 64135

Created attachment 100112 [details] create and remove the symlink like old-run-webkit-tests used to

Comment on attachment 100112 [details] create and remove the symlink like old-run-webkit-tests used to Please just fix the tests. This patch is wrong on several levels.

Ugh. I wonder why we haven't seen the mac NRWT port failling this test more often. Adam already R-'ed this, but at any rate, this is the wrong way to work around this. The top-level run_webkit_tests.py script is just a wrapper around controllers/manager.py which does all the real work. Assuming we did want NRWT to hack this, we should create the link (if it doesn't already exist) in manager.py:824 set_up_run(), and remove the link (if and only if we created it) in clean_up_run() in 951. However, we really shouldn't have to be doing this at all, and NRWT shouldn't need to be aware of this. Chromium, for example, is smart enough to dynamically remap this in pathToLocalResource(), and the other ports probably should be as well. Can you file a separate bug to get the DRT's fixed if we can't just fix the tests?

Comment on attachment 100112 [details] create and remove the symlink like old-run-webkit-tests used to Yeah, definitely the wrong layer. What tests depend on this?

(In reply to comment #4) > (From update of attachment 100112 [details]) > Yeah, definitely the wrong layer. What tests depend on this? There's a (probably not exhaustive) list at the first comment. Here they are: fast/dom/frame-loading-via-document-write.html fast/loader/local-CSS-from-local.html fast/loader/local-JavaScript-from-local.html fast/loader/local-image-from-local.html

Gonna fix this, it's bothering me as well. And no, this should not be fixed in the NRWT (especially not outside of the manager). I think the tests can be fixed instead.

Ok, so I don't want to lower the integrity of the fast/loader tests by making the urls relative (as opposed to fully qualified file:// scheme), which was my original intent. Similarly, those tests in http/tests/security which test loading local resources from a remote document still need the pathToLocalResource normalization. I'll open a ticket to fix the DRTs pathToLocalResource() so it normalizes file:///tmp/LayoutTests like Chromium's.

(In reply to comment #4) > (From update of attachment 100112 [details]) > Yeah, definitely the wrong layer. What tests depend on this? I'm not sure whether all these tests still exist, but in BUG 33153 I found a bunch of these which also cause problems when running multiple instances of run-webkit-tests with differing repositories. LayoutTests/fast/dom/frame-loading-via-document-write.html LayoutTests/fast/loader/local-CSS-from-local.html LayoutTests/fast/loader/local-JavaScript-from-local.html LayoutTests/fast/loader/local-iFrame-source-from-local.html LayoutTests/fast/loader/local-image-from-local.html LayoutTests/fast/loader/user-style-sheet-resource-load-callbacks.html LayoutTests/http/tests/misc/image-blocked-src-change.html LayoutTests/http/tests/misc/image-blocked-src-no-change.html LayoutTests/http/tests/security/frame-loading-via-document-write.html LayoutTests/http/tests/security/local-CSS-from-remote.html LayoutTests/http/tests/security/local-JavaScript-from-remote.html LayoutTests/http/tests/security/local-iFrame-from-remote.html LayoutTests/http/tests/security/local-image-from-remote.html LayoutTests/http/tests/security/local-user-CSS-from-remote.html LayoutTests/http/tests/security/local-video-poster-from-remote.html LayoutTests/http/tests/security/local-video-source-from-remote.html LayoutTests/http/tests/security/local-video-src-from-remote.html

Output of a "grep -r "pathToLocalResource(" * | awk '{ print $1 }'" on LayoutTests: fast/dom/frame-loading-via-document-write.html: fast/loader/local-image-from-local-expected.txt:Currently fast/loader/local-JavaScript-from-local-expected.txt:Currently fast/loader/local-iFrame-source-from-local.html: fast/loader/local-iFrame-source-from-local.html: fast/loader/local-image-from-local.html: fast/loader/local-image-from-local.html: fast/loader/local-iFrame-source-from-local-expected.txt:Currently fast/loader/local-CSS-from-local.html: fast/loader/local-CSS-from-local.html: fast/loader/local-CSS-from-local-expected.txt:Currently fast/loader/local-JavaScript-from-local.html: fast/loader/local-JavaScript-from-local.html: http/tests/security/local-CSS-from-remote.html: http/tests/security/local-CSS-from-remote.html: http/tests/security/local-video-poster-from-remote.html: http/tests/security/local-JavaScript-from-remote-expected.txt:Currently http/tests/security/local-JavaScript-from-remote.html: http/tests/security/local-JavaScript-from-remote.html: http/tests/security/local-CSS-from-remote-expected.txt:Currently http/tests/security/local-image-from-remote-expected.txt:Currently http/tests/security/local-iFrame-from-remote.html: http/tests/security/local-iFrame-from-remote.html: http/tests/security/local-video-source-from-remote.html: http/tests/security/local-image-from-remote.html: http/tests/security/local-image-from-remote.html: http/tests/security/frame-loading-via-document-write.html: http/tests/security/local-image-from-remote-whitelisted.html:var http/tests/security/local-iFrame-from-remote-expected.txt:Currently http/tests/security/local-video-src-from-remote.html: http/tests/plugins/post-url-file.html: Output of a "grep -r "/tmp/LayoutTests/" * | awk '{ print $1 }'" on LayoutTests: fast/dom/frame-loading-via-document-write.html: fast/loader/local-iFrame-source-from-local.html: fast/loader/user-style-sheet-resource-load-callbacks.html: fast/loader/local-image-from-local.html: fast/loader/local-CSS-from-local.html: fast/loader/local-JavaScript-from-local.html: http/tests/security/local-CSS-from-remote.html: http/tests/security/local-video-poster-from-remote.html: http/tests/security/local-user-CSS-from-remote.html: http/tests/security/local-JavaScript-from-remote.html: http/tests/security/local-iFrame-from-remote.html: http/tests/security/local-video-source-from-remote.html: http/tests/security/local-image-from-remote.html: http/tests/security/frame-loading-via-document-write.html: http/tests/security/local-image-from-remote-whitelisted.html:var http/tests/security/local-video-src-from-remote.html: http/tests/misc/image-blocked-src-no-change.html: http/tests/misc/image-blocked-src-no-change.html: http/tests/misc/image-blocked-src-change.html: