Mostly merging work by Peter Kelly. Reviewed by Maciej, landed by ap.
authorap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2006 20:16:03 +0000 (20:16 +0000)
committerap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2006 20:16:03 +0000 (20:16 +0000)
commitbf28fbfba48da44a6409e346e4b2f5af5984ca39
tree7ce00ae56b55e67273dc9ffc88890a83b0dc9776
parentc1954e89b3770a8ad983b98f2770562cbd0b2426
    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.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@12076 268f45cc-cd09-0410-ab3c-d52691b4dbfc
JavaScriptCore/ChangeLog
JavaScriptCore/kjs/array_object.cpp
JavaScriptCore/kjs/array_object.h
LayoutTests/ChangeLog
LayoutTests/fast/js/array-tostring-ignore-separator-expected.txt [new file with mode: 0644]
LayoutTests/fast/js/array-tostring-ignore-separator.html [new file with mode: 0644]
LayoutTests/fast/js/kde/Array-expected.txt
LayoutTests/fast/js/resources/array-tostring-ignore-separator.js [new file with mode: 0644]