LGTM
http://codereview.chromium.org/159703/diff/1/4 File src/debug.cc (right): http://codereview.chromium.org/159703/diff/1/4#newcode321 Line 321: // construct call. Please update this comment to reflect the CallFunction stub as well. http://codereview.chromium.org/159703/diff/1/4#newcode324 Line 324: if (code->is_call_stub()) { Please add a Handle here to avoid unexpected GC issues when changing the code (I know it is not your code). http://codereview.chromium.org/159703/diff/1/4#newcode339 Line 339: if (IsDebugBreak()) { And here. http://codereview.chromium.org/159703/diff/1/4#newcode1169 Line 1169: Object* obj = Heap::code_stubs()->SlowReverseLookup(*call_function_stub); Please use a Handle here even though not strictly required. http://codereview.chromium.org/159703/diff/1/4#newcode1190 Line 1190: JSFunction* js_function = JSFunction::cast(fun); And here. http://codereview.chromium.org/159703 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
