LGTM
http://codereview.chromium.org/1672006/diff/9001/10001 File src/liveedit-debugger.js (right): http://codereview.chromium.org/1672006/diff/9001/10001#newcode50 src/liveedit-debugger.js:50: // This API is a legacy and is obsolete. Why not remove it then? http://codereview.chromium.org/1672006/diff/9001/10001#newcode541 src/liveedit-debugger.js:541: info_node.new_end_pos = chunk_it.TranslatePos(info_node.info.end_position); Long line. http://codereview.chromium.org/1672006/diff/9001/10005 File test/mjsunit/debug-liveedit-newsource.js (right): http://codereview.chromium.org/1672006/diff/9001/10005#newcode64 test/mjsunit/debug-liveedit-newsource.js:64: assertEquals(25, something1); Maybe add a comment to why this is not 26. http://codereview.chromium.org/1672006/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
