[Webkit-unassigned] [Bug 23609] New: Do not append trailing slash to urls with no path for KURL::parse

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=23609

           Summary: Do not append trailing slash to urls with no path for
                    KURL::parse
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: treat at kde.org


The KURL class has recently been changed in revision r40242 to assert in the
KURL single argument constructors when the string passed in does not match the
result of the parse.  The theory is that KURL::parse method should be
idempotent.  However, the KURL::parse method appends a trailing slash to a
perfectly valid URL that has no path.  This results in triggering the ASSERT in
KURL ctor by passing in a string of "http://google.com" for instance.

Looking at the history it seems KURL has been adding this trailing slash since
WebKit was originally created from KHTML.  With this history I am not sure the
right thing to do is to remove this appending of the trailing slash, but if the
assert is to stay then this is what is required.


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.