LGTM
https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.cc File src/elements.cc (right): https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.cc#newcode179 src/elements.cc:179: virtual MaybeObject* AddElementsToFixedArray( The Chromium styleguide says that in function declarations (as opposed to calls), each argument should be on its own line unless all arguments fit on one line. https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.h File src/elements.h (right): https://chromiumcodereview.appspot.com/9605006/diff/1/src/elements.h#newcode83 src/elements.h:83: // Deleted an element in an object, returning a new elements backing store. s/Deleted/Deletes/ https://chromiumcodereview.appspot.com/9605006/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
