LGTM
http://codereview.chromium.org/21076/diff/1/5 File src/d8.js (right): http://codereview.chromium.org/21076/diff/1/5#newcode106 Line 106: if (event.eventType() == Debug.DebugEvent.Break) { How about a switch on the event type instead? http://codereview.chromium.org/21076/diff/1/2 File test/mjsunit/debug-compile-event.js (right): http://codereview.chromium.org/21076/diff/1/2#newcode1 Line 1: // Copyright 2008 the V8 project authors. All rights reserved. 2009 http://codereview.chromium.org/21076/diff/1/2#newcode35 Line 35: Why put spaces here and not before the function? http://codereview.chromium.org/21076 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
