Changeset 12076 in webkit for trunk/JavaScriptCore/kjs/array_object.h

Timestamp:

Jan 13, 2006, 12:16:03 PM (19 years ago)

Author:

ap

Message:

Mostly merging work by Peter Kelly. Reviewed by Maciej, landed by ap.

• fix ​http://bugzilla.opendarwin.org/show_bug.cgi?id=6261 Misc. array object fixes from KJS

JavaScriptCore:

• kjs/array_object.cpp: Don't treat 2^{32-1 as a real array index property. (ArrayInstance::getOwnPropertySlot): Ditto. (ArrayInstance::deleteProperty): Ditto. (ArrayInstance::put): Ditto. (ArrayInstance::propList): Added a FIXME comment. (ArrayInstance::put): Throw exception on trying to set invalid array length. (ArrayProtoFunc::callAsFunction): Do not use a separator argument when doing toString/toLocalString.}
• kjs/array_object.h: Added MAX_ARRAY_INDEX.

LayoutTests:

• fast/js/kde/Array-expected.txt: Updated for newly-passing tests.
• fast/js/array-tostring-ignore-separator.html: Added.
• fast/js/array-tostring-ignore-separator-expected.txt: Added.
• fast/js/resources/array-tostring-ignore-separator.js: Added.

File:

• trunk/JavaScriptCore/kjs/array_object.h (modified) (1 diff)

trunk/JavaScriptCore/kjs/array_object.h

@@ -51 +51 @@
   };
 
+  const unsigned MAX_ARRAY_INDEX = 0xFFFFFFFEu;
+
   class ArrayObjectImp : public InternalFunctionImp {
   public: