LGTM
http://codereview.chromium.org/2799037/diff/1/2 File src/debug-debugger.js (right): http://codereview.chromium.org/2799037/diff/1/2#newcode400 src/debug-debugger.js:400: if (IS_UNDEFINED(actual_position)) { Is this undefined check still needed after the change to runtime.cc? http://codereview.chromium.org/2799037/diff/1/2#newcode613 src/debug-debugger.js:613: var pos = %SetFunctionBreakPoint(func, source_position, break_point); Maybe pos -> actual_position as above http://codereview.chromium.org/2799037/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
