Changeset 34868 in webkit for trunk/JavaScriptCore/kjs/JSArray.cpp

Timestamp:

Jun 28, 2008, 10:38:37 PM (17 years ago)

Author:

Darin Adler

Message:

JavaScriptCore:

2008-06-28 Darin Adler <Darin Adler>

Reviewed by Sam and Cameron.

• fix ​https://bugs.webkit.org/show_bug.cgi?id=19805 Array.concat turns missing array elements into "undefined"

Test: fast/js/array-holes.html

• JavaScriptCore.exp: No longer export JSArray::getItem.

• kjs/ArrayPrototype.cpp: (KJS::arrayProtoFuncConcat): Changed to use getProperty instead of JSArray::getItem -- need to handle properties from the prototype chain instead of ignoring them.

• kjs/JSArray.cpp: Removed getItem.
• kjs/JSArray.h: Ditto.

WebCore:

2008-06-28 Darin Adler <Darin Adler>

Reviewed by Sam and Cameron.

• fix ​https://bugs.webkit.org/show_bug.cgi?id=19805 Array.concat turns missing array elements into "undefined"

• bridge/jni/jni_utility.cpp: (KJS::Bindings::convertArrayInstanceToJavaArray): Use get instead of getItem, since we always want to consider values from the prototypes when looking at JavaScript arrays.

LayoutTests:

2008-06-28 Darin Adler <Darin Adler>

Reviewed by Sam and Cameron.

• test for ​https://bugs.webkit.org/show_bug.cgi?id=19805 Array.concat turns missing array elements into "undefined"

• fast/js/array-holes-expected.txt: Added.
• fast/js/array-holes.html: Added.
• fast/js/resources/array-holes.js: Added.

File:

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

trunk/JavaScriptCore/kjs/JSArray.cpp

@@ -132 +132 @@
 }
 
-JSValue* JSArray::getItem(unsigned i) const
-{
-    ASSERT(i <= maxArrayIndex);
-
-    ArrayStorage* storage = m_storage;
-
-    if (i < storage->m_vectorLength) {
-        JSValue* value = storage->m_vector[i];
-        return value ? value : jsUndefined();
-    }
-
-    SparseArrayValueMap* map = storage->m_sparseValueMap;
-    if (!map)
-        return jsUndefined();
-
-    JSValue* value = map->get(i);
-    return value ? value : jsUndefined();
-}
-
 JSValue* JSArray::lengthGetter(ExecState* exec, const Identifier&, const PropertySlot& slot)
 {