LGTM. Just one nit, but feel free to move that over to one of your follow-up CLs.
https://chromiumcodereview.appspot.com/10441091/diff/1/src/d8.cc File src/d8.cc (right): https://chromiumcodereview.appspot.com/10441091/diff/1/src/d8.cc#newcode378 src/d8.cc:378: String::New("byteOffset out of bounds")); That should fit into one line. https://chromiumcodereview.appspot.com/10441091/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
