Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(700)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: src/runtime/runtime-debug.cc

Issue 2758483002: [debugger] tuned StepNext and StepOut at return position (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/inspector/v8-debugger-agent-impl.cc ('K') | « src/inspector/v8-debugger-agent-impl.cc ('k') | test/inspector/debugger/stepping-tail-call.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/runtime/runtime-debug.cc
diff --git a/src/runtime/runtime-debug.cc b/src/runtime/runtime-debug.cc
index 747548b14f7e8f469810e8c7b129434aae90ec0d..6e861cae69d56c9c0d45512ee1193b025ba6b442 100644
--- a/src/runtime/runtime-debug.cc
+++ b/src/runtime/runtime-debug.cc
@@ -1808,7 +1808,7 @@ RUNTIME_FUNCTION(Runtime_DebugOnFunctionCall) {
HandleScope scope(isolate);
DCHECK_EQ(1, args.length());
CONVERT_ARG_HANDLE_CHECKED(JSFunction, fun, 0);
- if (isolate->debug()->last_step_action() >= StepIn) {
+ if (isolate->debug()->last_step_action() != StepNone) {
isolate->debug()->PrepareStepIn(fun);
}
if (isolate->needs_side_effect_check() &&
« src/inspector/v8-debugger-agent-impl.cc ('K') | « src/inspector/v8-debugger-agent-impl.cc ('k') | test/inspector/debugger/stepping-tail-call.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698