RESOLVED FIXED 16487
KURL doesn't preserve isNull when constructed with a DeprecatedString 
https://bugs.webkit.org/show_bug.cgi?id=16487
Summary KURL doesn't preserve isNull when constructed with a DeprecatedString
Brett Wilson (Google)
Reported 2007-12-17 14:48:40 PST
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.
Attachments
Add attachment proposed patch, testcase, etc.
Note You need to log in before you can comment on or make changes to this bug.