LGTM

We are not linting the line lengths of the JS files, so I accepted the long
lines in d8.js.

Committed: http://code.google.com/p/v8/source/detail?r=6180

Thank you for the patch!

It would be nice if you could follow up with some tests of the new debug
protocol messages like we already have for the current messages in
test/mjsumit/debug-*

We also need to update http://code.google.com/p/v8/wiki/DebuggerProtocol with information on the new messages and changes to the existing ones. Actually you
will not be able to edit http://code.google.com/p/v8/wiki/DebuggerProtocol
online, but you can send a update and I will put it in.

http://codereview.chromium.org/5980006/

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

Reply via email to