LGTM
http://codereview.chromium.org/6499013/diff/1/test/cctest/test-api.cc File test/cctest/test-api.cc (right): http://codereview.chromium.org/6499013/diff/1/test/cctest/test-api.cc#newcode5640 test/cctest/test-api.cc:5640: v8::HandleScope scope1; No need for this since you have the handle scope above. http://codereview.chromium.org/6499013/diff/1/test/cctest/test-api.cc#newcode5648 test/cctest/test-api.cc:5648: ExpectTrue("Object.keys(other).indexOf('blocked_prop')==-1"); Spaces around '=='. http://codereview.chromium.org/6499013/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
