LGTM (with a nit).
https://chromiumcodereview.appspot.com/9117032/diff/2003/src/cpu-profiler.cc File src/cpu-profiler.cc (right): https://chromiumcodereview.appspot.com/9117032/diff/2003/src/cpu-profiler.cc#newcode43 src/cpu-profiler.cc:43: static const int kTickSamplesBufferChunkSize = 64*KB; I know this is not part of your change, but could you add spaces before and after the multiplication operator here? https://chromiumcodereview.appspot.com/9117032/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
