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

src/lookup.h

Index: src/lookup.h
diff --git a/src/lookup.h b/src/lookup.h
index be8394afec4e7242fd294695f6b31f347dac8628..48604b2389a375da7a99dd09bea37e217bfb3062 100644
--- a/src/lookup.h
+++ b/src/lookup.h
@@ -59,12 +59,10 @@
         holder_map_(holder_->map(), isolate_),
         initial_holder_(holder_),
         number_(DescriptorArray::kNotFound) {
-#if 0  // TODO(verwaest): Enable once blocking hacks are removed.
 #ifdef DEBUG
     uint32_t index;  // Assert that the name is not an array index.
     DCHECK(!name->AsArrayIndex(&index));
 #endif  // DEBUG
-#endif
     Next();
   }
 
@@ -85,12 +83,10 @@
         holder_map_(holder_->map(), isolate_),
         initial_holder_(holder_),
         number_(DescriptorArray::kNotFound) {
-#if 0  // TODO(verwaest): Enable once blocking hacks are removed.
 #ifdef DEBUG
     uint32_t index;  // Assert that the name is not an array index.
     DCHECK(!name->AsArrayIndex(&index));
 #endif  // DEBUG
-#endif
     Next();
   }
 
@@ -212,9 +208,7 @@
   Handle<Object> GetAccessors() const;
   Handle<InterceptorInfo> GetInterceptor() const;
   Handle<Object> GetDataValue() const;
-  // Usually returns the value that was passed in. In case of typed array
-  // accesses it returns the converted value.
-  Handle<Object> WriteDataValue(Handle<Object> value);
+  void WriteDataValue(Handle<Object> value);
   void InternalizeName();
 
  private: