LGTM (with a few nits).
https://chromiumcodereview.appspot.com/10558005/diff/1/src/d8.cc File src/d8.cc (right): https://chromiumcodereview.appspot.com/10558005/diff/1/src/d8.cc#newcode444 src/d8.cc:444: !args[0]->ToObject()->GetHiddenValue( Indentation seems off. https://chromiumcodereview.appspot.com/10558005/diff/1/src/d8.cc#newcode482 src/d8.cc:482: if (args[0]->IsObject() && Can we make this a proper "else if" so that all cases are on the same nesting level? https://chromiumcodereview.appspot.com/10558005/diff/1/src/d8.cc#newcode501 src/d8.cc:501: ARRAY_SIZE(buffer_args), buffer_args); Indentation seems off. https://chromiumcodereview.appspot.com/10558005/diff/1/src/d8.h File src/d8.h (right): https://chromiumcodereview.appspot.com/10558005/diff/1/src/d8.h#newcode398 src/d8.h:398: static Handle<Value> CreateExternalArray(const Arguments& args, Shouldn't this one actually return Handle<Object> as well? https://chromiumcodereview.appspot.com/10558005/diff/1/test/mjsunit/mjsunit.js File test/mjsunit/mjsunit.js (left): https://chromiumcodereview.appspot.com/10558005/diff/1/test/mjsunit/mjsunit.js#oldcode324 test/mjsunit/mjsunit.js:324: var actualConstructor = Object.prototypeOf(obj).constructor; How did that even ... ah forget it. https://chromiumcodereview.appspot.com/10558005/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
