LGTM

http://codereview.chromium.org/6673042/diff/2002/test/cctest/test-api.cc
File test/cctest/test-api.cc (right):

http://codereview.chromium.org/6673042/diff/2002/test/cctest/test-api.cc#newcode5717
test/cctest/test-api.cc:5717: v8::Handle<Value> value;
Add comment here saying something like: We follow Safari in ignoring
assignments to host object accessors.

http://codereview.chromium.org/6673042/

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

Reply via email to