Changeset 205507 in webkit for trunk/Source/JavaScriptCore/runtime/JSSet.cpp

Timestamp:

Sep 6, 2016, 2:35:24 PM (9 years ago)

Author:

Ryan Haddad

Message:

Unreviewed, rolling out r205504.
​https://bugs.webkit.org/show_bug.cgi?id=161645

Broke the iOS device build (Requested by ryanhaddad on
#webkit).

Reverted changeset:

"Make JSMap and JSSet faster"
​https://bugs.webkit.org/show_bug.cgi?id=160989
​http://trac.webkit.org/changeset/205504

Patch by Commit Queue <​[email protected]> on 2016-09-06

File:

• trunk/Source/JavaScriptCore/runtime/JSSet.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/runtime/JSSet.cpp

@@ -1 +1 @@
 /*
- * Copyright (C) 2013, 2016 Apple Inc. All rights reserved.
+ * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -27 +27 @@
 #include "JSSet.h"
 
-#include "JSCInlines.h"
+#include "CopiedBlockInlines.h"
+#include "JSCJSValueInlines.h"
+#include "JSSetIterator.h"
+#include "MapDataInlines.h"
+#include "SlotVisitorInlines.h"
+#include "StructureInlines.h"
 
 namespace JSC {
 
 const ClassInfo JSSet::s_info = { "Set", &Base::s_info, 0, CREATE_METHOD_TABLE(JSSet) };
+
+void JSSet::destroy(JSCell* cell)
+{
+    JSSet* thisObject = jsCast<JSSet*>(cell);
+    thisObject->JSSet::~JSSet();
+}
 
 String JSSet::toStringName(const JSObject*, ExecState*)
@@ -38 +49 @@
 }
 
+size_t JSSet::estimatedSize(JSCell* cell)
+{
+    JSSet* thisObject = jsCast<JSSet*>(cell);
+    size_t setDataSize = thisObject->m_setData.capacityInBytes();
+    return Base::estimatedSize(cell) + setDataSize;
 }
+
+void JSSet::visitChildren(JSCell* cell, SlotVisitor& visitor)
+{
+    Base::visitChildren(cell, visitor);
+    jsCast<JSSet*>(cell)->m_setData.visitChildren(cell, visitor);
+}
+
+void JSSet::copyBackingStore(JSCell* cell, CopyVisitor& visitor, CopyToken token)
+{
+    Base::copyBackingStore(cell, visitor, token);
+    jsCast<JSSet*>(cell)->m_setData.copyBackingStore(visitor, token);
+}
+
+bool JSSet::has(ExecState* exec, JSValue value)
+{
+    return m_setData.contains(exec, value);
+}
+
+size_t JSSet::size(ExecState* exec)
+{
+    return m_setData.size(exec);
+}
+
+void JSSet::add(ExecState* exec, JSValue value)
+{
+    m_setData.set(exec, this, value, value);
+}
+
+void JSSet::clear(ExecState*)
+{
+    m_setData.clear();
+}
+
+bool JSSet::remove(ExecState* exec, JSValue value)
+{
+    return m_setData.remove(exec, value);
+}
+
+}