Changeset 37458 in webkit for trunk/JavaScriptCore/kjs/StructureID.cpp

Timestamp:

Oct 9, 2008, 6:21:09 PM (17 years ago)

Author:

[email protected]

Message:

2008-10-09 Sam Weinig <​[email protected]>

Reviewed by Maciej Stachowiak.

Add leaks checking to StructureID.

• kjs/StructureID.cpp: (JSC::StructureID::StructureID): (JSC::StructureID::~StructureID):

File:

• trunk/JavaScriptCore/kjs/StructureID.cpp (modified) (4 diffs)

trunk/JavaScriptCore/kjs/StructureID.cpp

@@ -27 +27 @@
 #include "StructureID.h"
 
-#include "identifier.h"
 #include "JSObject.h"
 #include "PropertyNameArray.h"
+#include "identifier.h"
 #include "lookup.h"
+#include <wtf/RefCountedLeakCounter.h>
 #include <wtf/RefPtr.h>
 
@@ -36 +37 @@
 
 namespace JSC {
+
+#ifndef NDEBUG    
+static WTF::RefCountedLeakCounter structureIDCounter("StructureID");
+#endif
 
 StructureID::StructureID(JSValue* prototype, const TypeInfo& typeInfo)
@@ -51 +56 @@
     ASSERT(m_prototype);
     ASSERT(m_prototype->isObject() || m_prototype->isNull());
+
+#ifndef NDEBUG
+    structureIDCounter.increment();
+#endif
+}
+
+StructureID::~StructureID()
+{
+    if (m_previous) {
+        ASSERT(m_previous->m_transitionTable.contains(make_pair(m_nameInPrevious, m_attributesInPrevious)));
+        m_previous->m_transitionTable.remove(make_pair(m_nameInPrevious, m_attributesInPrevious));
+    }
+
+    if (m_cachedPropertyNameArrayData)
+        m_cachedPropertyNameArrayData->setCachedStructureID(0);
+
+#ifndef NDEBUG
+    structureIDCounter.decrement();
+#endif
 }
 
@@ -195 +219 @@
 }
 
-StructureID::~StructureID()
-{
-    if (m_previous) {
-        ASSERT(m_previous->m_transitionTable.contains(make_pair(m_nameInPrevious, m_attributesInPrevious)));
-        m_previous->m_transitionTable.remove(make_pair(m_nameInPrevious, m_attributesInPrevious));
-    }
-
-    if (m_cachedPropertyNameArrayData)
-        m_cachedPropertyNameArrayData->setCachedStructureID(0);
-}
-
 StructureIDChain* StructureID::createCachedPrototypeChain()
 {