LGTM with nits.
https://chromiumcodereview.appspot.com/10187001/diff/1/ChangeLog File ChangeLog (right): https://chromiumcodereview.appspot.com/10187001/diff/1/ChangeLog#newcode5 ChangeLog:5: ([email protected]) We don't need the email addresses in the ChangeLog. https://chromiumcodereview.appspot.com/10187001/diff/1/ChangeLog#newcode11 ChangeLog:11: Show names for the strong roots in heap snapshot. Is this important enough to be mentioned? https://chromiumcodereview.appspot.com/10187001/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
