LGTM, thanks!

http://codereview.chromium.org/596116/diff/1/2
File test/mjsunit/array-shift.js (right):

http://codereview.chromium.org/596116/diff/1/2#newcode68
test/mjsunit/array-shift.js:68: delete Array.prototype[3];
Why do you have these deletes here?  If you want them, you should test
if they return true or false.

http://codereview.chromium.org/596116

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

Reply via email to