Changeset 30413 in webkit

Timestamp:

Feb 19, 2008, 2:09:48 PM (18 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

Files:

• API/JSClassRef.cpp (modified) (4 diffs)
• API/JSClassRef.h (modified) (1 diff)
• API/JSObjectRef.cpp (modified) (1 diff)
• ChangeLog (modified) (1 diff)
• bindings/runtime_root.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();
 }
 

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-02-19  Anders Carlsson  <[email protected]>
+
+        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):
+
 2008-02-19  Darin Adler  <[email protected]>
 

trunk/JavaScriptCore/bindings/runtime_root.cpp

@@ -198 +198 @@
 PassRefPtr<RootObject> RootObject::create(const void* nativeHandle, JSGlobalObject* globalObject)
 {
-    return new RootObject(nativeHandle, globalObject);
+    return adoptRef(new RootObject(nativeHandle, globalObject));
 }
 
 RootObject::RootObject(const void* nativeHandle, JSGlobalObject* globalObject)
-    : RefCounted<RootObject>(0)
-    , m_isValid(true)
+    : m_isValid(true)
     , m_nativeHandle(nativeHandle)
     , m_globalObject(globalObject)