LGTM
http://codereview.chromium.org/113698/diff/9/1007 File src/debug.cc (right): http://codereview.chromium.org/113698/diff/9/1007#newcode2403 Line 2403: // Top::context() may have been NULL when the event occured. Please add "script collected" before event in comment. http://codereview.chromium.org/113698/diff/9/1006 File test/cctest/test-debug.cc (right): http://codereview.chromium.org/113698/diff/9/1006#newcode4849 Line 4849: // Count the number of breaks. breaks -> scripts collected http://codereview.chromium.org/113698/diff/9/1006#newcode4884 Line 4884: Heap::CollectAllGarbageIfContextDisposed(); Maybe use Heap::CollectAllGarbage(), instad of Heap::CollectAllGarbageIfContextDisposed() to avoid depending on the heuristics of the latter. http://codereview.chromium.org/113698/diff/9/1006#newcode4889 Line 4889: // Should SetMessageHandler2 cause execution of Debugger::UnloadDebugger like It should, but please remove this comment and open a bug instead. http://codereview.chromium.org/113698 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
