Changeset 35815 in webkit for trunk/JavaScriptCore/API/tests/testapi.c

Timestamp:

Aug 18, 2008, 1:09:59 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Maciej.

Make JSGlobalContextRetain/Release actually work.

• API/JSContextRef.cpp: (JSGlobalContextRetain): (JSGlobalContextRelease): Ref/deref global data to give checking for globalData.refCount() some sense.

• API/tests/testapi.c: (main): Added a test for this bug.

• kjs/JSGlobalData.cpp: (KJS::JSGlobalData::~JSGlobalData): While checking for memory leaks, found that JSGlobalData::emptyList has changed to a pointer, but it was not destructed, causing a huge leak in run-webkit-tests --threaded.

File:

• trunk/JavaScriptCore/API/tests/testapi.c (modified) (1 diff)

trunk/JavaScriptCore/API/tests/testapi.c

@@ -582 +582 @@
     JSClassRef globalObjectClass = JSClassCreate(&globalObjectClassDefinition);
     context = JSGlobalContextCreate(globalObjectClass);
-    
+
+    JSGlobalContextRetain(context);
+    JSGlobalContextRelease(context);
+
     JSObjectRef globalObject = JSContextGetGlobalObject(context);
     ASSERT(JSValueIsObject(context, globalObject));