Skip to content
  • ggaren's avatar
    JavaScriptCore: · 3917acb0
    ggaren authored
            Reviewed by Sam Weinig.
            
            Added JSObject::removeDirect, to support the fix for 
            <rdar://problem/5522487> REGRESSION: With JavaScript disabled, any 
            page load causes a crash in PropertyMap::put
    
            * kjs/object.cpp:
            (KJS::JSObject::removeDirect):
            * kjs/object.h:
    
    WebCore:
    
            Reviewed by Sam Weinig.
            
            New fix for <rdar://problem/5522487> REGRESSION: With JavaScript 
            disabled, any page load causes a crash in PropertyMap::put
            
            Explicitly remove the "document" property from the window. The old 
            solution would leave a stale "document" property around after JavaScript
            was re-enabled.
    
            The architecture for disabling JavaScript could use some consolidation. 
            It seems wrong that a script proxy even exists when JavaScript is 
            disabled. It also seems wrong that so many individual call sites are 
            responsible for checking whether JavaScript is enabled. I've filed a 
            bug about this: http://bugs.webkit.org/show_bug.cgi?id=15385.
    
            * bindings/js/kjs_proxy.cpp:
            (WebCore::KJSProxy::clearDocumentWrapper):
            * bindings/js/kjs_proxy.h:
            * page/Frame.cpp:
            (WebCore::Frame::setDocument):
    
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@26074 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    3917acb0