[Webkit-unassigned] [Bug 16487] New: KURL doesn't preserve isNull when constructed with a DepricatedString

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=16487

           Summary: KURL doesn't preserve isNull when constructed with a
                    DepricatedString
           Product: WebKit
           Version: 525+ (Nightly build)
          Platform: PC
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: marv.decker at gmail.com


This means for this code:
  KURL empty;
  KURL second(empty.url());
then empty.isNull() != second.isNull().

This is the pattern executed in DocLoader::requestResource when it does
completeURL(). See bug 16480 for this extra string->KURL conversion.

The issue is that if completeURL generated an empty KURL, the urlString will be
NULL (as in fast/loader/simultaneous-reloads-assert.html where the base is a
data: URL), and then completeURL will convert it to a [Depricated]String, which
will then be used to create a new KURL that is now empty but not NULL.

This masks bug 16485, where DocLoader::checkForReload will crash if the string
isNull.


-- 
Configure bugmail: http://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.