LGTM
http://codereview.chromium.org/2868046/diff/5001/6002 File src/runtime.cc (right): http://codereview.chromium.org/2868046/diff/5001/6002#newcode2443 src/runtime.cc:2443: if (subject->IsAsciiRepresentation() || subject->HasOnlyAsciiChars()) { Does subject->IsAsciiRepresentation() imply subject->HasOnlyAsciiChars()? (I.e., could we do with just HasOnlyAsciiChars?) http://codereview.chromium.org/2868046/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
