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

Unified Diff: test/mjsunit/debug-stepout-recursive-function.js

Issue 1525173003: [debugger] remove step count parameter from prepare step. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years 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
Index: test/mjsunit/debug-stepout-recursive-function.js
diff --git a/test/mjsunit/debug-stepout-recursive-function.js b/test/mjsunit/debug-stepout-recursive-function.js
index 3741f26a8d055fc94e2c11ca23f7105ab28803a3..9082294b7827594ff40c4db4d52367c534e0f5f8 100644
--- a/test/mjsunit/debug-stepout-recursive-function.js
+++ b/test/mjsunit/debug-stepout-recursive-function.js
@@ -30,7 +30,6 @@
Debug = debug.Debug
var exception = null;
-var step_out_count = 1;
// Simple debug event handler which counts the number of breaks hit and steps.
var break_point_hit_count = 0;
@@ -40,7 +39,7 @@ function listener(event, exec_state, event_data, data) {
break_point_hit_count++;
// Continue stepping until returned to bottom frame.
if (exec_state.frameCount() > 1) {
- exec_state.prepareStep(Debug.StepAction.StepOut, step_out_count);
+ exec_state.prepareStep(Debug.StepAction.StepOut);
}
}
@@ -80,27 +79,23 @@ function fact(x) {
BeginTest('Test 1');
shouldBreak = function(x) { return x == 3; };
-step_out_count = 1;
fact(3);
EndTest(2);
BeginTest('Test 2');
shouldBreak = function(x) { return x == 2; };
-step_out_count = 1;
fact(3);
EndTest(3);
BeginTest('Test 3');
shouldBreak = function(x) { return x == 1; };
-step_out_count = 2;
fact(3);
-EndTest(2);
+EndTest(4);
BeginTest('Test 4');
shouldBreak = function(x) { return x == 1 || x == 3; };
-step_out_count = 2;
fact(3);
-EndTest(3);
+EndTest(5);
// Get rid of the debug event listener.
Debug.setListener(null);

Powered by Google App Engine
This is Rietveld 408576698