I think this needs to be thought trough by people writing the spec. This is just an ad-hoc workaround, which will likely not match whatever is later decided as
the actual spec.

If you really need to get something in *right now* for whatever reason, I'd only
sign off on *only* changing runtime-array.cc/IsConcatSpreadable to include:
if (obj->IsJSObject() && !isolate->MayAccess(Handle<JSObject>::cast(obj))) {
  // or whatever value is the default non-spreadable value.
  return isolate->heap()->false_value();
}

https://codereview.chromium.org/1230793002/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to