Bug 37700 – Test file: URL canonicalization

• Component: New Bugs
• Status: RESOLVED FIXED

[Maciej Stachowiak]

Test file: URL canonicalization

[Maciej Stachowiak]

Created attachment 53518 [details]
Patch

[Adam Barth]

Comment on attachment 53518 [details]
Patch

+ sustem

typo

Otherwise looks good.  Hopefully we won't need to end up with platform-specific behavior.

[Maciej Stachowiak]

Committed r57719: <http://trac.webkit.org/changeset/57719>

[Adam Barth]

This was rolled out because it broke some windows bots.  I'm making a new version now.

[Adam Barth]

Created attachment 53601 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 53601 [details]
Patch for landing

Rejecting patch 53601 from commit-queue.

Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--exit-after-n-failures=1', '--quiet']" exit_code: 1
Running build-dumprendertree
Compiling Java tests
make: Nothing to be done for `default'.
Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests
Testing 12694 test cases.
fast/url/file.html -> failed

Exiting early after 1 failures. 8879 tests run.
158.72s total testing time

8878 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
4 test cases (<1%) had stderr output

Full output: http://webkit-commit-queue.appspot.com/results/1717131

[Adam Barth]

Created attachment 53607 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 53607 [details]
Patch for landing

Clearing flags on attachment: 53607

Committed r57786: <http://trac.webkit.org/changeset/57786>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[James Robinson]

Looks like this fails on the Chromium canaries, I think the diffs are only rebaselines.  Here's what the results look like on chromium+win:


Canonicalization of file URLs

On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".


FAIL canonicalize('file:c:\\foo\\bar.html') should be file:///C:/foo/bar.html. Was file:///tmp/mock/c:/foo/bar.html.
FAIL canonicalize('  File:c|////foo\\bar.html') should be file:///C:////foo/bar.html. Was file:///tmp/mock/c%7C////foo/bar.html.
FAIL canonicalize('file:') should be file:///. Was file:///tmp/mock/path.
FAIL canonicalize('file:UNChost/path') should be file://unchost/path. Was file:///tmp/mock/UNChost/path.
FAIL canonicalize('c:\\foo\\bar') should be file:///C:/foo/bar. Was c:\foo\bar.
FAIL canonicalize('C|/foo/bar') should be file:///C:/foo/bar. Was file:///tmp/mock/C%7C/foo/bar.
FAIL canonicalize('/C|\\foo\\bar') should be file:///C:/foo/bar. Was file:///C%7C/foo/bar.
FAIL canonicalize('//C|/foo/bar') should be file:///C:/foo/bar. Was file://c%7C/foo/bar.
PASS canonicalize('//server/file') is 'file://server/file'
PASS canonicalize('\\\\server\\file') is 'file://server/file'
PASS canonicalize('/\\server/file') is 'file://server/file'
FAIL canonicalize('file:c:foo/bar.html') should be file:///C:/foo/bar.html. Was file:///tmp/mock/c:foo/bar.html.
PASS canonicalize('file:/\\/\\C:\\\\//foo\\bar.html') is 'file:///C:////foo/bar.html'
PASS canonicalize('file:///foo/bar.txt') is 'file:///foo/bar.txt'
FAIL canonicalize('FILE:/\\/\\7:\\\\//foo\\bar.html') should be file://7:////foo/bar.html. Was file:///7:////foo/bar.html.
FAIL canonicalize('file:filer/home\\me') should be file://filer/home/me. Was file:///tmp/mock/filer/home/me.
FAIL canonicalize('file:///C:/foo/../../../bar.html') should be file:///C:/bar.html. Was file:///bar.html.
FAIL canonicalize('file:///C:/asdf#\xc2') should be file:///C:/asdf#�. Was file:///C:/asdf#Â.
PASS canonicalize('file:///home/me') is 'file:///home/me'
FAIL canonicalize('file:c:\\foo\\bar.html') should be file:///c:/foo/bar.html. Was file:///tmp/mock/c:/foo/bar.html.
FAIL canonicalize('file:c|//foo\\bar.html') should be file:///c%7C//foo/bar.html. Was file:///tmp/mock/c%7C//foo/bar.html.
PASS canonicalize('//') is 'file:///'
PASS canonicalize('///') is 'file:///'
PASS canonicalize('///test') is 'file:///test'
PASS canonicalize('file://test') is 'file://test/'
PASS canonicalize('file://localhost') is 'file://localhost/'
PASS canonicalize('file://localhost/') is 'file://localhost/'
PASS canonicalize('file://localhost/test') is 'file://localhost/test'
PASS successfullyParsed is true

TEST COMPLETE

I'll land new baselines into platform/chromium.  It is a bummer to have to maintain two sets of results for these.

[Adam Barth]

> I'll land new baselines into platform/chromium.  It is a bummer to have to
> maintain two sets of results for these.

Indeed.  That's why we're working to unfork the URL parser.  :)