Bug 76619 - [WK2] fast/url/degenerate-file-base.html fails
Summary: [WK2] fast/url/degenerate-file-base.html fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Alexey Proskuryakov
URL:
Keywords: Qt, QtTriaged
Depends on: 76699
Blocks: 76496
  Show dependency tree / graph
 
Reported: 2012-01-19 04:04 PST by Csaba Osztrogonác
Modified: 2012-01-20 04:30 PST (History)
5 users (show)

See Also:

Attachments
proposed fix (2.90 KB, patch)
2012-01-19 11:05 PST, Alexey Proskuryakov 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2012-01-19 04:04:49 PST 
This test introduced in http://trac.webkit.org/changeset/105294, but 
fails on Qt-WK2. I don't know if it is Qt specific bug or WK2 specific 
bug, because the Apple SL WK2 tester doesn't work long long time ago.

--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/url/degenerate-file-base-expected.txt 
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/fast/url/degenerate-file-base-actual.txt 
@@ -1,2 +1,2 @@
 Base URL: file:///
-Should say PASS: PASS
+Should say PASS:
Comment 1 Csaba Osztrogonác 2012-01-19 05:05:39 PST 
I skipped it: http://trac.webkit.org/changeset/105405/trunk/LayoutTests/platform/qt-wk2/Skipped
Comment 2 Alexey Proskuryakov 2012-01-19 10:27:35 PST 
It also fails on Mac WK2. Looking into this.
Comment 3 Alexey Proskuryakov 2012-01-19 11:00:08 PST 
This is caused by a bug in WebKitTestRunner - it creates a wrong URL for UNIX style paths. Note the two slashes after localhost:

file://localhost//Users/ap/Safari/OpenSource/LayoutTests/fast/url/degenerate-file-base.html
Comment 4 Alexey Proskuryakov 2012-01-19 11:05:32 PST 
Created attachment 123149 [details]
proposed fix

There is a good chance that this will change many other WK2 test results. I didn't check locally, because I'm getting way too much noise from already failing tests.
Comment 5 WebKit Review Bot 2012-01-19 16:13:21 PST 
Comment on attachment 123149 [details]
proposed fix

Clearing flags on attachment: 123149

Committed r105461: <http://trac.webkit.org/changeset/105461>
Comment 6 WebKit Review Bot 2012-01-19 16:13:26 PST 
All reviewed patches have been landed.  Closing bug.