Changeset 32648 in webkit for trunk/JavaScriptCore/kjs

Timestamp:

Apr 28, 2008, 9:54:24 AM (17 years ago)

Author:

Darin Adler

Message:

2008-04-28 Darin Adler <Darin Adler>

Reviewed by Mitz.

• fix assertion hit every time you view www.apple.com

• kjs/PropertyNameArray.cpp: (KJS::PropertyNameArray::add): Changed assertion to allow null and empty strings. Now to find out why we have a property named "" and if that's a bug!

File:

• trunk/JavaScriptCore/kjs/PropertyNameArray.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/PropertyNameArray.cpp

@@ -29 +29 @@
 void PropertyNameArray::add(UString::Rep* identifier)
 {
-    ASSERT(identifier->identifierTable);
+    ASSERT(identifier == &UString::Rep::null || identifier == &UString::Rep::empty || identifier->identifierTable);
 
     size_t size = m_vector.size();