Reland of Replace SetObjectProperty / DefineObjectProperty with less powerful alternatives where re…

src/lookup.cc

Index: src/lookup.cc
diff --git a/src/lookup.cc b/src/lookup.cc
index 1bb491a9dee2aa528f1a85de8a9756f99c8143bf..6a26663b44f87380dfaf9efb5ed16b57467e35ca 100644
--- a/src/lookup.cc
+++ b/src/lookup.cc
@@ -419,12 +419,12 @@ Handle<Object> LookupIterator::GetDataValue() const {
 }
 
 
-Handle<Object> LookupIterator::WriteDataValue(Handle<Object> value) {
+void LookupIterator::WriteDataValue(Handle<Object> value) {
   DCHECK_EQ(DATA, state_);
   Handle<JSObject> holder = GetHolder<JSObject>();
   if (IsElement()) {
     ElementsAccessor* accessor = holder->GetElementsAccessor();
-    return accessor->Set(holder, index_, value);
+    accessor->Set(holder, index_, value);
   } else if (holder->IsGlobalObject()) {
     Handle<GlobalDictionary> property_dictionary =
         handle(holder->global_dictionary());
@@ -439,7 +439,6 @@ Handle<Object> LookupIterator::WriteDataValue(Handle<Object> value) {
   } else {
     DCHECK_EQ(v8::internal::DATA_CONSTANT, property_details_.type());
   }
-  return value;
 }