LGTM
http://codereview.chromium.org/608006/diff/7001/7002 File src/api.cc (right): http://codereview.chromium.org/608006/diff/7001/7002#newcode2467 src/api.cc:2467: // Records that the given string's characters were copied to some Could you update the comments in this class to make it clear that you do not have a counter per string? http://codereview.chromium.org/608006/diff/7001/7003 File test/cctest/test-api.cc (right): http://codereview.chromium.org/608006/diff/7001/7003#newcode474 test/cctest/test-api.cc:474: CHECK(small_string->CanMakeExternal()); small_ -> large_ http://codereview.chromium.org/608006/diff/7001/7003#newcode511 test/cctest/test-api.cc:511: CHECK(small_string->CanMakeExternal()); small_ -> large_ http://codereview.chromium.org/608006 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
