[Webkit-unassigned] [Bug 28151] New: KURL hasFragmentIdentifier()/removeFragmentIdentifier() should be cleaned up and made a little smarter

[bugzilla-daemon at webkit.org]

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

           Summary: KURL
                    hasFragmentIdentifier()/removeFragmentIdentifier()
                    should be cleaned up and made a little smarter
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P3
         Component: Page Loading
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: beidson at apple.com


KURL hasFragmentIdentifier()/removeFragmentIdentifier() should be cleaned up
and made a little smarter.

There's a handful of callsites that always say:
if (someURL.hasFragmentIdentifier())
    someURL.removeFragmentIdentifier();

Which is kind of silly and might be cleaner as one line of code...
...except that removeFragmentIdentifier() always allocs a new string and
reparses it, when it should really do its own "if I don't have a fragment ID,
don't do anything" check.

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