LGTM

http://codereview.chromium.org/1584008/diff/2001/3001
File src/debug-debugger.js (right):

http://codereview.chromium.org/1584008/diff/2001/3001#newcode2008
src/debug-debugger.js:2008: invocation();
nit: alignment

http://codereview.chromium.org/1584008/diff/2001/3002
File src/liveedit-debugger.js (right):

http://codereview.chromium.org/1584008/diff/2001/3002#newcode446
src/liveedit-debugger.js:446: new_source.substring(diff.change_pos,
diff.change_pos + diff.new_len),
nit: you could use 'substr' method, which accepts length as its second
argument.

http://codereview.chromium.org/1584008

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

To unsubscribe, reply using "remove me" as the subject.

Reply via email to