LGTM (if comment is addressed).
https://codereview.chromium.org/11274047/diff/1/src/proxy.js File src/proxy.js (right): https://codereview.chromium.org/11274047/diff/1/src/proxy.js#newcode65 src/proxy.js:65: %InstallFunctions($Proxy, DONT_ENUM, [ Change this to not use a natives syntax runtime call. Also you could prepend a call to %CheckIsBootstrapping() and put it into a separate setup method. That would be in line with the way we set up other builtin objects. https://codereview.chromium.org/11274047/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
