LGTM (with nits).
http://codereview.chromium.org/8349031/diff/1/test/mjsunit/define-properties.js File test/mjsunit/define-properties.js (right): http://codereview.chromium.org/8349031/diff/1/test/mjsunit/define-properties.js#newcode29 test/mjsunit/define-properties.js:29: // modifications are done. Can we add this whole test case to object-define-properties.js instead of adding a new file? http://codereview.chromium.org/8349031/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
