Move reconfiguration into the elements accessor

src/lookup.cc

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/v8.h"
 
 #include "src/bootstrapper.h"
 #include "src/deoptimizer.h"
 #include "src/lookup.h"
 #include "src/lookup-inl.h"
@@ skipping 138 matching lines @@
   ReloadPropertyInformation();
 }
 
 
 void LookupIterator::ReconfigureDataProperty(Handle<Object> value,
                                              PropertyAttributes attributes) {
   DCHECK(state_ == DATA || state_ == ACCESSOR);
   DCHECK(HolderIsReceiverOrHiddenPrototype());
   Handle<JSObject> holder = GetHolder<JSObject>();
   if (IsElement()) {
-    // TODO(verwaest): Clean up.
-    DCHECK(holder->HasFastElements() || holder->HasDictionaryElements() ||
-           holder->HasSloppyArgumentsElements());
+    DCHECK(!holder->HasExternalArrayElements());
+    DCHECK(!holder->HasFixedTypedArrayElements());
     DCHECK(attributes != NONE || !holder->HasFastElements());
-    Handle<SeededNumberDictionary> d = JSObject::NormalizeElements(holder);
-    // TODO(verwaest): Move this into NormalizeElements.
-    d->set_requires_slow_elements();
-    if (holder->HasDictionaryElements()) {
-      JSObject::SetDictionaryElement(holder, index(), value, attributes);
-    } else {
-      JSObject::SetDictionaryArgumentsElement(holder, index(), value,
-                                              attributes);
-    }
+    Handle<FixedArrayBase> elements(holder->elements());
+    holder->GetElementsAccessor()->Reconfigure(holder, elements, number_, value,
+                                               attributes);
   } else if (holder_map_->is_dictionary_map()) {
     PropertyDetails details(attributes, v8::internal::DATA, 0,
                             PropertyCellType::kMutable);
     JSObject::SetNormalizedProperty(holder, name(), value, details);
   } else {
     holder_map_ = Map::ReconfigureExistingProperty(
         holder_map_, descriptor_number(), i::kData, attributes);
     holder_map_ =
         Map::PrepareForDataProperty(holder_map_, descriptor_number(), value);
     JSObject::MigrateToMap(holder, holder_map_);
@@ skipping 300 matching lines @@
       case InterceptorState::kSkipNonMasking:
         return true;
       case InterceptorState::kProcessNonMasking:
         return false;
     }
   }
   return interceptor_state_ == InterceptorState::kProcessNonMasking;
 }
 }  // namespace internal
 }  // namespace v8