LGTM with comment.
https://chromiumcodereview.appspot.com/10823202/diff/2001/src/extensions/statistics-extension.cc File src/extensions/statistics-extension.cc (right): https://chromiumcodereview.appspot.com/10823202/diff/2001/src/extensions/statistics-extension.cc#newcode142 src/extensions/statistics-extension.cc:142: "amount_of_external_allocated_memory"); It seems to me that a macro would save a lot of space since Size(), Available() and CommittedMemory() are used repeatedly. https://chromiumcodereview.appspot.com/10823202/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
