Changeset 93913 in webkit for trunk/Source/JavaScriptCore/runtime/JSGlobalObject.h

Timestamp:

Aug 26, 2011, 2:40:45 PM (14 years ago)

Author:

[email protected]

Message:

Get rid of frame life support timer
​https://bugs.webkit.org/show_bug.cgi?id=66874

Reviewed by Geoff Garen. V8 fixes courtesy of Dmitry Titov.

Test: fast/frames/detached-frame-property.html

• bindings/js/JSDOMWindowBase.cpp:
• bindings/js/JSDOMWindowBase.h:
• bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld):
• bindings/v8/ScriptController.cpp: (WebCore::ScriptController::evaluate):
• bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::runScript): (WebCore::V8Proxy::callFunction): (WebCore::V8Proxy::newInstance):
• page/Frame.cpp: (WebCore::Frame::Frame): (WebCore::Frame::~Frame): (WebCore::Frame::pageDestroyed):
• page/Frame.h:
• page/Page.cpp: (WebCore::Page::~Page): Removed all code that dealt with the life support timer. Protect Frame inside runScript() and callFunction() for v8 bindings.

• bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent): Replaced ref/deref calls on self with a RefPtr.

File:

• trunk/Source/JavaScriptCore/runtime/JSGlobalObject.h (modified) (1 diff)

trunk/Source/JavaScriptCore/runtime/JSGlobalObject.h

@@ -263 +263 @@
         virtual bool isGlobalObject() const { return true; }
 
-        virtual ExecState* globalExec();
+        ExecState* globalExec();
 
         virtual bool shouldInterruptScript() const { return true; }