Added IsProperty() check to result in DefineOrRedefineAccessor - if no check
authorricow@chromium.org <ricow@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Thu, 4 Feb 2010 11:28:03 +0000 (11:28 +0000)
committerricow@chromium.org <ricow@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Thu, 4 Feb 2010 11:28:03 +0000 (11:28 +0000)
commit4f179b09ba620c18db674adc094cd56dbc90e5f3
tree715411677d70719f05519be66ce14a72e0777189
parentf0ef4d7bac877cedf7796db21cc53ef57be962ca
Added IsProperty() check to result in DefineOrRedefineAccessor - if no check
this causes the debug check to fails since type() asserts that the
lookup_type != NOT_FOUND. This does not change any functionality since we
explicitly checked if it was one of the three types that we need to delete.
Also changed defineProperties in v8natives to actually return the object
being defined (as specified by spec)

Review URL: http://codereview.chromium.org/572003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@3794 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
src/runtime.cc
src/v8natives.js
test/mjsunit/object-define-properties.js [new file with mode: 0644]