Reviewers: Rico, Message: Hello Rico, it looks like we've both been working on DefineOwnProperty. Does this look reasonable?
Description: First cut at bug 992 Fixes JS portion of DefineOwnProperty when there is an existing property and the new descriptor is generic. Makes code follow spec steps more closely. Fixes typo for check for unchanged enumerable in step 6. Adds regression test. BUG= TEST= Please review this at http://codereview.chromium.org/6035014/ SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge Affected files: M src/runtime.cc M src/v8natives.js M test/mjsunit/object-define-property.js A test/mjsunit/regress/regress-992.js -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
