LGTM with an additional test case.
http://codereview.chromium.org/6902144/diff/1/src/array.js File src/array.js (left): http://codereview.chromium.org/6902144/diff/1/src/array.js#oldcode132 src/array.js:132: if (!IS_UNDEFINED(e)) { Is this a bug fix or does it change the behavior? We should have a test case hitting this to verify that this is the right thing to do. http://codereview.chromium.org/6902144/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
