LGTM

http://codereview.chromium.org/2827033/diff/1/2
File test/es5conform/es5conform.status (right):

http://codereview.chromium.org/2827033/diff/1/2#newcode230
test/es5conform/es5conform.status:230: # Wront test - the test in the
end should be i === true.
"Wrong" (or, preferably, "Bad").
"in the end" -> "at the end".
Quotes around "i === true".

http://codereview.chromium.org/2827033/diff/1/2#newcode245
test/es5conform/es5conform.status:245: # Wrong tests, path in test file
is wrong. This will crash the test
"Bad tests - "

http://codereview.chromium.org/2827033/diff/1/2#newcode250
test/es5conform/es5conform.status:250: # Wrong test - deleting the
property on o in callbackfn will
Bad.

http://codereview.chromium.org/2827033/show

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

Reply via email to