LGTM

http://codereview.chromium.org/548045/diff/8/2002
File src/debug.h (right):

http://codereview.chromium.org/548045/diff/8/2002#newcode657
src/debug.h:657: // TODO(peter.rybin): should we check that we pending
unload is still actual?
If you want a TODO in the source please create a bug for it and use the
form TODO(bug).

http://codereview.chromium.org/548045/diff/8/2002#newcode659
src/debug.h:659: // Or else Debug::ClearMirrorCache() will fail in first
line.
You could add an ASSERT that the debugger is not entered.

http://codereview.chromium.org/548045
-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to