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.