Reviewers: rossberg,
https://codereview.chromium.org/11369151/diff/1/src/array.js File src/array.js (right): https://codereview.chromium.org/11369151/diff/1/src/array.js#newcode444 src/array.js:444: delete this[n]; This change is required for proper operation, and is the proper ordering per the spec: http://ecma-international.org/ecma-262/5.1/#sec-15.4.4.6 Description: Minimal implementation and tests of observable array methods Bail out of any special-casing in array methods. Further optimization is possible, but can be left for later. Please review this at https://codereview.chromium.org/11369151/ SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge Affected files: M src/array.js M src/builtins.cc M test/mjsunit/harmony/object-observe.js -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
