https://chromiumcodereview.appspot.com/10128006/diff/1/src/profile-generator.h File src/profile-generator.h (right):
https://chromiumcodereview.appspot.com/10128006/diff/1/src/profile-generator.h#newcode1028 src/profile-generator.h:1028: const char* GetGcSubrootName(Object* object); If it deals only with strong roots, perhaps call it 'GetStrongGcSubrootName'? https://chromiumcodereview.appspot.com/10128006/diff/1/src/profile-generator.h#newcode1043 src/profile-generator.h:1043: HashMap gc_subroot_names_; Why are you using a generic HashMap? HeapObjectsSet can associate heap objects with tags. https://chromiumcodereview.appspot.com/10128006/diff/1/test/cctest/test-heap-profiler.cc File test/cctest/test-heap-profiler.cc (right): https://chromiumcodereview.appspot.com/10128006/diff/1/test/cctest/test-heap-profiler.cc#newcode13 test/cctest/test-heap-profiler.cc:13: #include <ctype.h> System headers must go before project headers: http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Names_and_Order_of_Includes#Names_and_Order_of_Includes https://chromiumcodereview.appspot.com/10128006/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
