Changeset 101945 in webkit for trunk/Source/JavaScriptCore/API/JSClassRef.cpp

Timestamp:

Dec 3, 2011, 9:20:08 PM (14 years ago)

Author:

Darin Adler

Message:

Use HashMap<OwnPtr> for the opaqueJSClassData map
​https://bugs.webkit.org/show_bug.cgi?id=73759

Reviewed by Andreas Kling.

• API/JSClassRef.cpp:

(OpaqueJSClass::contextData): Update types.

• runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::~JSGlobalData): Add an explicit clear of opaqueJSClassData to keep the
timing the same. If we didn't care about the order of operations, we could remove this, too.

• runtime/JSGlobalData.h: Use OwnPtr instead of raw pointer for the mapped type in the

opaqueJSClassData map.

File:

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

trunk/Source/JavaScriptCore/API/JSClassRef.cpp

@@ -203 +203 @@
 OpaqueJSClassContextData& OpaqueJSClass::contextData(ExecState* exec)
 {
-    OpaqueJSClassContextData*& contextData = exec->globalData().opaqueJSClassData.add(this, 0).first->second;
+    OwnPtr<OpaqueJSClassContextData>& contextData = exec->globalData().opaqueJSClassData.add(this, nullptr).first->second;
     if (!contextData)
-        contextData = new OpaqueJSClassContextData(exec->globalData(), this);
+        contextData = adoptPtr(new OpaqueJSClassContextData(exec->globalData(), this));
     return *contextData;
 }