On 2014/03/04 21:43:41, arv wrote:
Let me know if you want me to split this into two patches?


https://codereview.chromium.org/187053003/diff/30001/test/mjsunit/regress/regress-3135.js
File test/mjsunit/regress/regress-3135.js (right):


https://codereview.chromium.org/187053003/diff/30001/test/mjsunit/regress/regress-3135.js#newcode28
test/mjsunit/regress/regress-3135.js:28:
assertEquals('{"__proto__":{"__proto__":{"__proto__":null},"x":7},"x":8}',
This surprised me at first so I had to dig deep into the spec. This is the
expected result because with a replacer array we end up doing a [[Get]]. This
matches Chakra and SpiderMonkey.

LGTM.

https://codereview.chromium.org/187053003/

--
--
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/groups/opt_out.

Reply via email to