Changeset 100672 in webkit for trunk/Source/JavaScriptCore/runtime/JSActivation.cpp

Timestamp:

Nov 17, 2011, 2:04:07 PM (14 years ago)

Author:

[email protected]

Message:

Add finalizer to JSActivation
​https://bugs.webkit.org/show_bug.cgi?id=72575

Reviewed by Geoffrey Garen.

• runtime/JSActivation.cpp:

(JSC::JSActivation::finishCreation): Attach finalize function to objects during creation.
(JSC::JSActivation::finalize):

• runtime/JSActivation.h: Replaced virtual destructor with static finalize function.

File:

• trunk/Source/JavaScriptCore/runtime/JSActivation.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/runtime/JSActivation.cpp

@@ -57 +57 @@
     // doesn't know about SharedSymbolTable
     static_cast<SharedSymbolTable*>(m_symbolTable)->ref();
-}
-
-JSActivation::~JSActivation()
-{
-    static_cast<SharedSymbolTable*>(m_symbolTable)->deref();
+    callFrame->globalData().heap.addFinalizer(this, &finalize);
+}
+
+void JSActivation::finalize(JSCell* cell)
+{
+    static_cast<SharedSymbolTable*>(jsCast<JSActivation*>(cell)->m_symbolTable)->deref();
 }