Changeset 30413 in webkit for trunk/JavaScriptCore/API

Timestamp:

Feb 19, 2008, 2:09:48 PM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Darin.

Change OpaqueJSClass and RootObject to start with a ref count of 1.

• API/JSClassRef.cpp: (OpaqueJSClass::OpaqueJSClass): (OpaqueJSClass::createNoAutomaticPrototype): (OpaqueJSClass::create):
• API/JSClassRef.h:
• API/JSObjectRef.cpp: (JSClassCreate):
• bindings/runtime_root.cpp: (KJS::Bindings::RootObject::create): (KJS::Bindings::RootObject::RootObject):

Location:

trunk/JavaScriptCore/API

Files:

• JSClassRef.cpp (modified) (4 diffs)
• JSClassRef.h (modified) (1 diff)
• JSObjectRef.cpp (modified) (1 diff)

trunk/JavaScriptCore/API/JSClassRef.cpp

@@ -41 +41 @@
 OpaqueJSClass::OpaqueJSClass(const JSClassDefinition* definition, OpaqueJSClass* protoClass) 
     // FIXME: <rdar://problem/4949018>
-    : RefCounted<OpaqueJSClass>(0)
-    , className(definition->className)
+    : className(definition->className)
     , parentClass(definition->parentClass)
     , prototypeClass(0)
@@ -100 +99 @@
 }
 
-JSClassRef OpaqueJSClass::createNoAutomaticPrototype(const JSClassDefinition* definition)
+PassRefPtr<OpaqueJSClass> OpaqueJSClass::createNoAutomaticPrototype(const JSClassDefinition* definition)
 {
-    return new OpaqueJSClass(definition, 0);
+    return adoptRef(new OpaqueJSClass(definition, 0));
 }
 
@@ -112 +111 @@
 }
 
-JSClassRef OpaqueJSClass::create(const JSClassDefinition* definition)
+PassRefPtr<OpaqueJSClass> OpaqueJSClass::create(const JSClassDefinition* definition)
 {
     if (const JSStaticFunction* staticFunctions = definition->staticFunctions) {
@@ -124 +123 @@
         JSClassDefinition objectDefinition = *definition;
         objectDefinition.staticFunctions = 0;
-        return new OpaqueJSClass(&objectDefinition, protoClass);
+        return adoptRef(new OpaqueJSClass(&objectDefinition, protoClass));
     }
 
-    return new OpaqueJSClass(definition, 0);
+    return adoptRef(new OpaqueJSClass(definition, 0));
 }
 

trunk/JavaScriptCore/API/JSClassRef.h

@@ -58 +58 @@
 
 struct OpaqueJSClass : public RefCounted<OpaqueJSClass> {
-    static OpaqueJSClass* create(const JSClassDefinition*);
-    static OpaqueJSClass* createNoAutomaticPrototype(const JSClassDefinition*);
+    static PassRefPtr<OpaqueJSClass> create(const JSClassDefinition*);
+    static PassRefPtr<OpaqueJSClass> createNoAutomaticPrototype(const JSClassDefinition*);
     ~OpaqueJSClass();
     

trunk/JavaScriptCore/API/JSObjectRef.cpp

@@ -50 +50 @@
 {
     JSLock lock;
-    JSClassRef jsClass = (definition->attributes & kJSClassAttributeNoAutomaticPrototype)
+    RefPtr<OpaqueJSClass> jsClass = (definition->attributes & kJSClassAttributeNoAutomaticPrototype)
         ? OpaqueJSClass::createNoAutomaticPrototype(definition)
         : OpaqueJSClass::create(definition);
     
-    return JSClassRetain(jsClass);
+    return jsClass.release().releaseRef();
 }