Changeset 104119 in webkit for trunk/Source/JavaScriptCore/runtime/Arguments.h

Timestamp:

Jan 4, 2012, 11:39:36 PM (13 years ago)

Author:

[email protected]

Message:

<​http://webkit.org/b/75604> All instances of JSC::ArgumentsData appear to be leaked by JSC::Arguments

Since JSC::Arguments has an OwnPtr for a member it needs to override destroy
to ensure that the correct destructor is invoked. This is necessary because
JSCell subclasses all intentionally have non-virtual destructors.

Reviewed by Filip Pizlo.

• runtime/Arguments.cpp:

(JSC::Arguments::destroy):

• runtime/Arguments.h:

File:

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

trunk/Source/JavaScriptCore/runtime/Arguments.h

@@ -109 +109 @@
 
     private:
+        static void destroy(JSCell*);
         static bool getOwnPropertySlot(JSCell*, ExecState*, const Identifier& propertyName, PropertySlot&);
         static bool getOwnPropertySlotByIndex(JSCell*, ExecState*, unsigned propertyName, PropertySlot&);