Overall looks good, but I have a question.
https://chromiumcodereview.appspot.com/10037004/diff/9001/test/cctest/test-heap-profiler.cc File test/cctest/test-heap-profiler.cc (right): https://chromiumcodereview.appspot.com/10037004/diff/9001/test/cctest/test-heap-profiler.cc#newcode632 test/cctest/test-heap-profiler.cc:632: "parsed.nodes.concat(0, 0, 0, 0, 0, 0, parsed.edges.length);"); And why do we need this line? https://chromiumcodereview.appspot.com/10037004/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
