Changeset 35853 in webkit for trunk/JavaScriptCore/API/JSBase.cpp

Timestamp:

Aug 20, 2008, 12:23:06 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Geoff Garen.

Bring back shared JSGlobalData and implicit locking, because too many clients rely on it.

File:

• trunk/JavaScriptCore/API/JSBase.cpp (modified) (4 diffs)

trunk/JavaScriptCore/API/JSBase.cpp

@@ -35 +35 @@
 #include <kjs/interpreter.h>
 #include <kjs/JSGlobalObject.h>
+#include <kjs/JSLock.h>
 #include <kjs/JSObject.h>
 
@@ -43 +44 @@
     ExecState* exec = toJS(ctx);
     exec->globalData().heap->registerThread();
+    JSLock lock(exec);
 
     JSObject* jsThisObject = toJS(thisObject);
@@ -67 +69 @@
     ExecState* exec = toJS(ctx);
     exec->globalData().heap->registerThread();
+    JSLock lock(exec);
 
     Completion completion = Interpreter::checkSyntax(exec->dynamicGlobalObject()->globalExec(), sourceURL->ustring(), startingLineNumber, script->ustring());
@@ -92 +95 @@
     Heap* heap = globalData.heap;
 
+    JSLock lock(globalData.isSharedInstance);
+
     if (!heap->isBusy())
         heap->collect();