Changeset 35293 in webkit for trunk/JavaScriptCore/API/JSContextRef.cpp

Timestamp:

Jul 23, 2008, 3:15:16 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Geoff Garen.

JSClassRef is created context-free, but gets infatuated with the first context it sees.

The implicit API contract is that JSClassRef can be used with any context on any thread.
This no longer worked, because UStrings in the class were turned into per-context
identifiers, and the cached JSObject prototype was tied to JSGlobalData, too.

File:

• trunk/JavaScriptCore/API/JSContextRef.cpp (modified) (1 diff)

trunk/JavaScriptCore/API/JSContextRef.cpp

@@ -52 +52 @@
 
     JSGlobalObject* globalObject = new (sharedGlobalData) JSCallbackObject<JSGlobalObject>(globalObjectClass);
-    JSGlobalContextRef ctx = toGlobalRef(globalObject->globalExec());
-    JSValue* prototype = globalObjectClass->prototype(ctx);
+    ExecState* exec = globalObject->globalExec();
+    JSValue* prototype = globalObjectClass->prototype(exec);
     if (!prototype)
         prototype = jsNull();
     globalObject->reset(prototype);
-    return JSGlobalContextRetain(ctx);
+    return JSGlobalContextRetain(toGlobalRef(exec));
 }