LGTM
Consider renaming "own" to something else, like no_prototype_check or
omit_prototype


http://codereview.chromium.org/6101001/diff/1/src/api.cc
File src/api.cc (right):

http://codereview.chromium.org/6101001/diff/1/src/api.cc#newcode1
src/api.cc:1:
Unintentional extra line?

http://codereview.chromium.org/6101001/diff/1/test/mjsunit/bugs/bug-1015.js
File test/mjsunit/bugs/bug-1015.js (right):

http://codereview.chromium.org/6101001/diff/1/test/mjsunit/bugs/bug-1015.js#newcode44
test/mjsunit/bugs/bug-1015.js:44: // in loop.
Indention

http://codereview.chromium.org/6101001/diff/1/test/mjsunit/bugs/bug-1015.js#newcode53
test/mjsunit/bugs/bug-1015.js:53: }
Add newline

http://codereview.chromium.org/6101001/diff/1/test/mjsunit/bugs/bug-1015.js#newcode62
test/mjsunit/bugs/bug-1015.js:62: bar: 10 };
indention

http://codereview.chromium.org/6101001/

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

Reply via email to