Allow getters and setters on JSArray elements.

src/objects.cc

Index: src/objects.cc
===================================================================
--- src/objects.cc	(revision 6157)
+++ src/objects.cc	(working copy)
@@ -2932,7 +2932,6 @@
 
   uint32_t index = 0;
   bool is_element = name->AsArrayIndex(&index);
-  if (is_element && IsJSArray()) return Heap::undefined_value();
 
   if (is_element) {
     switch (GetElementsKind()) {
@@ -6945,7 +6944,7 @@
   FixedArray* elms = FixedArray::cast(elms_obj);
   uint32_t elms_length = static_cast<uint32_t>(elms->length());
 
-  if (check_prototype && !IsJSArray() &&
+  if (check_prototype &&
       (index >= elms_length || elms->get(index)->IsTheHole())) {
     if (SetElementWithCallbackSetterInPrototypes(index, value)) {

[Lasse Reichstein, 2011/01/04 13:34:34]

For the heck of it, combine the two ifs into one.

[Rico, 2011/01/04 13:57:50]

Done.

       return value;
@@ -7080,7 +7079,7 @@
         }
       } else {
         // Index not already used. Look for an accessor in the prototype chain.
-        if (check_prototype && !IsJSArray() &&
+        if (check_prototype &&
             SetElementWithCallbackSetterInPrototypes(index, value)) {
           return value;
         }