Skip to content
  • darin's avatar
    LayoutTests: · 9d783fab
    darin authored
            - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=9176
              REGRESSION: repro crash in WebCore::StringImpl::hash() const + 28 (StringImpl.h:67)
    
            * fast/loader/link-no-URL.html: Added.
            * fast/loader/link-no-URL-expected.txt: Added.
    
    WebCore:
    
            Reviewed by Maciej.
    
            - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9176
              REGRESSION: repro crash in WebCore::StringImpl::hash() const + 28 (StringImpl.h:67)
    
            Test: fast/loader/link-no-URL.html
    
            * loader/Cache.h:
            * loader/Cache.cpp:
            (WebCore::Cache::updateCacheStatus): Removed the URL parameter. Instead get the URL
            from the cached object itself. Callers were passing in the wrong URL. When it was a
            null string, it caused the crash mentioned above. But in other cases it may have
            caused some other minor problems as well. Added an assert to catch cases where the
            URL is null (should never happen).
            (WebCore::Cache::requestImage): Changed call site to not pass URL.
            (WebCore::Cache::requestStyleSheet): Ditto.
            (WebCore::Cache::requestScript): Ditto.
            (WebCore::Cache::requestXSLStyleSheet): Ditto.
            (WebCore::Cache::requestXBLDocument): Ditto.
    
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@14732 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    9d783fab