Changeset 36789 in webkit for trunk/JavaScriptCore/VM/JSPropertyNameIterator.cpp

Timestamp:

Sep 22, 2008, 7:44:09 PM (17 years ago)

Author:

[email protected]

Message:

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

Reviewed by Maciej Stachowiak.

Patch for ​https://bugs.webkit.org/show_bug.cgi?id=21014
Speed up for..in by using StructureID to avoid call hasProperty

Speeds up fasta by 8%.

• VM/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::invalidate):
• VM/JSPropertyNameIterator.h: (JSC::JSPropertyNameIterator::next):
• kjs/PropertyNameArray.h: (JSC::PropertyNameArrayData::begin): (JSC::PropertyNameArrayData::end): (JSC::PropertyNameArrayData::setCachedStructureID): (JSC::PropertyNameArrayData::cachedStructureID):
• kjs/StructureID.cpp: (JSC::StructureID::getEnumerablePropertyNames): (JSC::structureIDChainsAreEqual):
• kjs/StructureID.h:

File:

• trunk/JavaScriptCore/VM/JSPropertyNameIterator.cpp (modified) (1 diff)

trunk/JavaScriptCore/VM/JSPropertyNameIterator.cpp

@@ -83 +83 @@
 void JSPropertyNameIterator::invalidate()
 {
+    ASSERT(m_position == m_end);
     m_object = 0;
     m_data.clear();