LGTM
https://chromiumcodereview.appspot.com/10078015/diff/1/test/cctest/test-heap-profiler.cc File test/cctest/test-heap-profiler.cc (right): https://chromiumcodereview.appspot.com/10078015/diff/1/test/cctest/test-heap-profiler.cc#newcode5 test/cctest/test-heap-profiler.cc:5: #include <ctype.h> Sorry, didn't spot that in your original CL. "v8.h" is a "project header", thus it goes after system headers. https://chromiumcodereview.appspot.com/10078015/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
