Changeset 181867 in webkit for trunk/Source/JavaScriptCore/runtime/Structure.cpp

Timestamp:

Mar 23, 2015, 1:48:18 PM (10 years ago)

Author:

Joseph Pecoraro

Message:

Fix DO_PROPERTYMAP_CONSTENCY_CHECK enabled build
​https://bugs.webkit.org/show_bug.cgi?id=142952

Reviewed by Geoffrey Garen.

• runtime/Structure.cpp:

(JSC::PropertyTable::checkConsistency):
The check offset method doesn't exist in PropertyTable, it exists in Structure.

(JSC::Structure::checkConsistency):
So move it here, and always put it at the start to match normal behavior.

File:

• trunk/Source/JavaScriptCore/runtime/Structure.cpp (modified) (4 diffs)

trunk/Source/JavaScriptCore/runtime/Structure.cpp

@@ -829 +829 @@
 #endif
 
-#if !DO_PROPERTYMAP_CONSTENCY_CHECK
-
-inline void Structure::checkConsistency()
-{
-    checkOffsetConsistency();
-}
-
-#endif
-
 PropertyTable* Structure::copyPropertyTable(VM& vm)
 {
@@ -1134 +1125 @@
 void PropertyTable::checkConsistency()
 {
-    checkOffsetConsistency();
     ASSERT(m_indexSize >= PropertyTable::MinimumTableSize);
     ASSERT(m_indexMask);
@@ -1192 +1182 @@
 void Structure::checkConsistency()
 {
+    checkOffsetConsistency();
+
     if (!propertyTable())
         return;
@@ -1203 +1195 @@
 
     propertyTable()->checkConsistency();
+}
+
+#else
+
+inline void Structure::checkConsistency()
+{
+    checkOffsetConsistency();
 }