[Webkit-unassigned] [Bug 22664] New: Rename and review uses of parseURL()

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Dec 4 16:52:16 PST 2008


           Summary: Rename and review uses of parseURL()
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Macintosh
        OS/Version: Mac OS X 10.5
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: ddkilzer at webkit.org
                CC: darin at apple.com

Bug 11850 Comment #4 quoting Darin Adler:

Here's what parseURL does:

    1) Strip leading and trailing spaces.
    2) Strip URL() if it surrounds the string, allowing any mix of uppercase
and lowercase.
    3) Strip leading and trailing spaces.
    4) Strip balanced single or double quotes.
    5) Strip leading and trailing spaces.
    6) Strip all characters in the range U+0000-U+000C

It's very strange that we want to do this anywhere except in CSS. In fact, I
think it's a bug almost every where this function is used. Perhaps some of this
is needed but obviously (2) is something specific to CSS property values even
if some of the other behavior is useful.

Someone should create some tests cases and investigate the behavior of other
web browsers for the various places we call this function. And at the same time
we should probably rename it.

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.

More information about the webkit-unassigned mailing list