Changeset 52334 in webkit for trunk/JavaScriptCore/API/JSClassRef.cpp

Timestamp:

Dec 18, 2009, 12:46:08 PM (15 years ago)

Author:

[email protected]

Message:

Fixed intermittent failure seen on Windows buildbot, and in other JSC
API clients.

Reviewed by Adam Roben.

Added a WeakGCPtr class and changed OpaqueJSClass::cachedPrototype to
use it, to avoid vending a stale object as a prototype.

• API/JSClassRef.cpp:

(OpaqueJSClassContextData::OpaqueJSClassContextData):
(OpaqueJSClass::prototype):

• API/JSClassRef.h: Use WeakGCPtr.

• JavaScriptCore.xcodeproj/project.pbxproj:
• runtime/WeakGCPtr.h: Added.

(JSC::WeakGCPtr::WeakGCPtr):
(JSC::WeakGCPtr::get):
(JSC::WeakGCPtr::clear):
(JSC::WeakGCPtr::operator*):
(JSC::WeakGCPtr::operator->):
(JSC::WeakGCPtr::operator!):
(JSC::WeakGCPtr::operator bool):
(JSC::WeakGCPtr::operator UnspecifiedBoolType):
(JSC::WeakGCPtr::assign):
(JSC::::operator):
(JSC::operator==):
(JSC::operator!=):
(JSC::static_pointer_cast):
(JSC::const_pointer_cast):
(JSC::getPtr): Added WeakGCPtr to the project.

File:

• trunk/JavaScriptCore/API/JSClassRef.cpp (modified) (2 diffs)

trunk/JavaScriptCore/API/JSClassRef.cpp

@@ -143 +143 @@
 OpaqueJSClassContextData::OpaqueJSClassContextData(OpaqueJSClass* jsClass)
     : m_class(jsClass)
-    , cachedPrototype(0)
 {
     if (jsClass->m_staticValues) {
@@ -241 +240 @@
         }
     }
-    return jsClassData.cachedPrototype;
-}
+    return jsClassData.cachedPrototype.get();
+}