Bug 53848 - Safari should lowercase schemes in cannonicalized urls to match every other browser
Summary: Safari should lowercase schemes in cannonicalized urls to match every other b...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Eric Seidel (no email)
URL:
Keywords:
: 16409 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-02-05 03:37 PST by Eric Seidel (no email)
Modified: 2011-04-29 11:35 PDT (History)
3 users (show)

See Also:


Attachments
Patch (7.45 KB, patch)
2011-02-05 03:38 PST, Eric Seidel (no email)
abarth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2011-02-05 03:37:30 PST
Safari should lowercase schemes in cannonicalized urls to match every other browser
Comment 1 Eric Seidel (no email) 2011-02-05 03:38:51 PST
Created attachment 81354 [details]
Patch
Comment 2 Adam Barth 2011-02-05 03:40:51 PST
Comment on attachment 81354 [details]
Patch

Did you perf test it?  :)
Comment 3 Adam Barth 2011-02-05 03:41:00 PST
+ap
Comment 4 Eric Seidel (no email) 2011-02-05 03:43:57 PST
Committed r77741: <http://trac.webkit.org/changeset/77741>
Comment 5 Eric Seidel (no email) 2011-02-05 04:48:49 PST
Committed r77743: <http://trac.webkit.org/changeset/77743>
Comment 6 Eric Seidel (no email) 2011-02-05 04:52:22 PST
I meant to use the commit-queue to land this, but forgot to. :(  That would have caught my missing expectation updates.

HOpefully all is well now.  Off to bed with me.
Comment 7 Alexey Proskuryakov 2011-04-29 11:35:03 PDT
*** Bug 16409 has been marked as a duplicate of this bug. ***