Map v8::Object to v8::internal::JSReceiver

src/accessors.cc

Index: src/accessors.cc
diff --git a/src/accessors.cc b/src/accessors.cc
index 1871822d617663ee9a38fe6db1df6b5ff100f235..8d3c9654b0070b98f0682149919f53b17baa17db 100644
--- a/src/accessors.cc
+++ b/src/accessors.cc
@@ -161,10 +161,12 @@ void Accessors::ArgumentsIteratorSetter(
     const v8::PropertyCallbackInfo<void>& info) {
   i::Isolate* isolate = reinterpret_cast<i::Isolate*>(info.GetIsolate());
   HandleScope scope(isolate);
-  Handle<JSObject> object_handle = Utils::OpenHandle(*info.This());
+  Handle<JSObject> object_handle =
+      Handle<JSObject>::cast(Utils::OpenHandle(*info.This()));
   Handle<Object> value_handle = Utils::OpenHandle(*val);
   Handle<Name> name_handle = Utils::OpenHandle(*name);
 
+  // TODO(jochen): Update to JSReceiver.

[Toon Verwaest, 2015/11/02 09:41:26]

These methods are guaranteed to be called directly on the known receiver type,
so there's not need to call out to JSReceiver:: here. The cast above is valid.

   if (JSObject::DefinePropertyOrElementIgnoreAttributes(
           object_handle, name_handle, value_handle, NONE)
           .is_null()) {
@@ -205,7 +207,7 @@ void Accessors::ArrayLengthSetter(
   i::Isolate* isolate = reinterpret_cast<i::Isolate*>(info.GetIsolate());
   HandleScope scope(isolate);
 
-  Handle<JSObject> object = Utils::OpenHandle(*info.This());
+  Handle<JSReceiver> object = Utils::OpenHandle(*info.This());
   Handle<JSArray> array = Handle<JSArray>::cast(object);
   Handle<Object> length_obj = Utils::OpenHandle(*val);