LGTM
I had not considered the possibility of being able to change writable to
false
even with configurable set to false. I saw the discussion on es-discuss and
that
is a somehow valid point (although it intuitively does not make a lot of
sense).
But we should of course follow the spec here.
http://codereview.chromium.org/6035014/diff/10001/src/v8natives.js
File src/v8natives.js (right):
http://codereview.chromium.org/6035014/diff/10001/src/v8natives.js#newcode645
src/v8natives.js:645: // Step 12 - updating an existing accessor
property with generic
with generic -> with a generic
http://codereview.chromium.org/6035014/
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
