LGTM
http://codereview.chromium.org/7388011/diff/2002/src/mirror-debugger.js File src/mirror-debugger.js (right): http://codereview.chromium.org/7388011/diff/2002/src/mirror-debugger.js#newcode1609 src/mirror-debugger.js:1609: for (var r = receiver; !r.isNull() && property.isUndefined(); r = r.protoObject()) { nit: While we're here let's fix the long line. http://codereview.chromium.org/7388011/diff/2002/test/mjsunit/debug-receiver.js File test/mjsunit/debug-receiver.js (right): http://codereview.chromium.org/7388011/diff/2002/test/mjsunit/debug-receiver.js#newcode93 test/mjsunit/debug-receiver.js:93: BeginTest("strict: 1"); TestStrict(1); EndTest(); Add heap number? http://codereview.chromium.org/7388011/diff/2002/test/mjsunit/debug-receiver.js#newcode116 test/mjsunit/debug-receiver.js:116: BeginTest("non-strict: 1"); TestNonStrict(1); EndTest(); Ditto. http://codereview.chromium.org/7388011/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
