Added object-define-properties.js test file to include test of return value from
Object.defineProperties (+ some extra tests)

http://codereview.chromium.org/572003/diff/2001/3002
File src/runtime.cc (right):

http://codereview.chromium.org/572003/diff/2001/3002#newcode2909
src/runtime.cc:2909: if (result.IsProperty() &&
On 2010/02/04 09:57:01, Mads Ager wrote:
I think you should use result.IsValid() here - no reason to check
which type of
valid result it is more than once.

Done.

http://codereview.chromium.org/572003

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

Reply via email to