Thanks Vitaly!

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()) {
On 2011/07/18 13:04:50, Vitaly Repeshko wrote:
nit: While we're here let's fix the long line.

Done.

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();
On 2011/07/18 13:04:50, Vitaly Repeshko wrote:
Add heap number?

Done.

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();
On 2011/07/18 13:04:50, Vitaly Repeshko wrote:
Ditto.

Done.

http://codereview.chromium.org/7388011/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to