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

Unified Diff: test/mjsunit/debug-stepframe.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-stepframe.js
diff --git a/test/mjsunit/debug-stepframe.js b/test/mjsunit/debug-stepframe.js
index f7983c010e07a8425d88ae8beb85330a943c83e6..8d9f959e78756c422c5845ac446a63dce6d244a1 100644
--- a/test/mjsunit/debug-stepframe.js
+++ b/test/mjsunit/debug-stepframe.js
@@ -82,7 +82,6 @@ Object.defineProperty(o, "set", { set : set });
Debug = debug.Debug;
var break_count = 0
var exception = null;
-var step_size;
function listener(event, exec_state, event_data, data) {
if (event != Debug.DebugEvent.Break) return;
@@ -92,22 +91,20 @@ function listener(event, exec_state, event_data, data) {
var match = line.match(/\/\/ Break (\d+)$/);
assertEquals(2, match.length);
assertEquals(break_count, parseInt(match[1]));
- break_count += step_size;
- exec_state.prepareStep(Debug.StepAction.StepFrame, step_size);
+ break_count ++;
+ exec_state.prepareStep(Debug.StepAction.StepFrame);
} catch (e) {
print(e + e.stack);
exception = e;
}
}
-for (step_size = 1; step_size < 6; step_size++) {
- print("step size = " + step_size);
- break_count = 0;
- Debug.setListener(listener);
- debugger; // Break 0
- f0();
- Debug.setListener(null); // Break 16
- assertTrue(break_count > 14);
-}
+
+break_count = 0;
+Debug.setListener(listener);
+debugger; // Break 0
+f0();
+Debug.setListener(null); // Break 16
+assertTrue(break_count > 14);
assertNull(exception);

Powered by Google App Engine
This is Rietveld 408576698