Bug 76496 – file:// doesn't work as base URL

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Alexey Proskuryakov]

KURL normalizes "file://" to "file:". When later used as a base URL, it's seen as a non-hierarchical one, making completion fail.

This works fine in both IE and Firefox, as the URL is canonicalized to "file:///".

[Alexey Proskuryakov]

Created attachment 122847 [details]
proposed patch

[Alexey Proskuryakov]

Committed <http://trac.webkit.org/changeset/105294>.

[Csaba Osztrogonác]

Reopen, because it broke tests:
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28Tests%29/r105294%20%2836456%29/results.html
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28Tests%29/r105294%20%2836456%29/http/tests/misc/bubble-drag-events-pretty-diff.html
http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r105294%20(32592)/fast/loader/url-parse-1-pretty-diff.html
http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r105294%20(32592)/http/tests/misc/iframe-invalid-source-crash-pretty-diff.html

Could you check and fix them?

[Alexey Proskuryakov]

Looking. Strange, I updated results of two of these.

[Alexey Proskuryakov]

Thank you - update results in <http://trac.webkit.org/changeset/105307>.

However, iframe-invalid-source-crash.html and bubble-drag-events.html appear to be broken on Qt because of a DumpRenderTree difference, where it dumps "file:///" as empty path instead of "/". I'll file a separate bug for this.