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

Unified Diff: test/mjsunit/debug-step-4.js

Issue 1525993002: [debugger] remove frame argument for prepare step. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: rebase 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
« no previous file with comments | « test/cctest/test-debug.cc ('k') | test/mjsunit/debug-step-4-in-frame.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/debug-step-4.js
diff --git a/test/mjsunit/debug-step-4-in-frame.js b/test/mjsunit/debug-step-4.js
similarity index 77%
rename from test/mjsunit/debug-step-4-in-frame.js
rename to test/mjsunit/debug-step-4.js
index 93884303ca78ce65345de0db3bdc6a8a861dedcc..c6b21af316751c4f1d40a38978cc949205c213c5 100644
--- a/test/mjsunit/debug-step-4-in-frame.js
+++ b/test/mjsunit/debug-step-4.js
@@ -53,8 +53,8 @@ function h() {
}
}
-function TestCase(frame_index, step_count, expected_final_state) {
- print("Test case, parameters " + frame_index + "/" + step_count);
+function TestCase(step_count, expected_final_state) {
+ print("Test case, step count: " + step_count);
var listener_exception = null;
var state_snapshot;
@@ -68,12 +68,7 @@ function TestCase(frame_index, step_count, expected_final_state) {
if (event == Debug.DebugEvent.Break) {
if (listener_state == 0) {
Debug.clearBreakPoint(bp);
- var context_frame;
- if (frame_index !== undefined) {
- context_frame = exec_state.frame(frame_index);
- }
- exec_state.prepareStep(Debug.StepAction.StepNext,
- step_count, context_frame);
+ exec_state.prepareStep(Debug.StepAction.StepNext, step_count);
listener_state = 1;
} else if (listener_state == 1) {
state_snapshot = String(state);
@@ -107,26 +102,8 @@ function TestCase(frame_index, step_count, expected_final_state) {
// Warm-up -- make sure all is compiled and ready for breakpoint.
h();
-
-// Stepping in the default (top) frame.
-TestCase(undefined, 0, "0,0,-1");
-TestCase(undefined, 1, "0,0,-1");
-TestCase(undefined, 2, "0,0,0");
-TestCase(undefined, 5, "0,0,1");
-TestCase(undefined, 8, "0,0,3");
-
-// Stepping in the frame #0 (should be exactly the same as above).
-TestCase(0, 0, "0,0,-1");
-TestCase(0, 1, "0,0,-1");
-TestCase(0, 2, "0,0,0");
-TestCase(0, 5, "0,0,1");
-TestCase(0, 8, "0,0,3");
-
-// Stepping in the frame #1.
-TestCase(1, 0, "0,0,3");
-TestCase(1, 3, "0,1,3");
-TestCase(1, 7, "0,3,3");
-
-// Stepping in the frame #2.
-TestCase(2, 3, "1,3,3");
-TestCase(2, 7, "3,3,3");
+TestCase(0, "0,0,-1");
+TestCase(1, "0,0,-1");
+TestCase(2, "0,0,0");
+TestCase(5, "0,0,1");
+TestCase(8, "0,0,2");
« no previous file with comments | « test/cctest/test-debug.cc ('k') | test/mjsunit/debug-step-4-in-frame.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698