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