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

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

Issue 26703009: Revert "Debug: Allow stepping into on a given call frame." (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 2 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
« no previous file with comments | « src/runtime.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/debug-step-4-in-frame.js
diff --git a/test/mjsunit/debug-step-4-in-frame.js b/test/mjsunit/debug-step-4-in-frame.js
index 483883adc6475231c6d165a4d0cb0f749a334c31..65ac4902dd9f0863ce894863fdc168cb813bc600 100644
--- a/test/mjsunit/debug-step-4-in-frame.js
+++ b/test/mjsunit/debug-step-4-in-frame.js
@@ -29,17 +29,12 @@
// Get the Debug object exposed from the debug context global object.
Debug = debug.Debug
-// Tests how debugger can step into/over not necessarily in the top frame.
+// Tests how debugger can step over not necessarily in the top frame.
// Simple 3 functions, that protocol their execution state in global
// variable state.
var state;
-function e() {
- for (state[3] = 0; state[3] < 5; state[3]++) {
- void ("" + e + state[3]);
- }
-}
function f() {
var a = 1978;
for (state[2] = 0; state[2] < 5; state[2]++) {
@@ -48,20 +43,18 @@ function f() {
}
function g() {
for (state[1] = 0; state[1] < 5; state[1]++) {
- f() + e();
+ f();
}
}
function h() {
- state = [-1, -1, -1, -1];
+ state = [-1, -1, -1];
for (state[0] = 0; state[0] < 5; state[0]++) {
g();
}
}
-function TestCase(frame_index, step_count, expected_final_state, action) {
- action = action || Debug.StepAction.StepNext;
- print("Test case, parameters " + frame_index + "/" + step_count +
- ", action " + action);
+function TestCase(frame_index, step_count, expected_final_state) {
+ print("Test case, parameters " + frame_index + "/" + step_count);
var listener_exception = null;
var state_snapshot;
@@ -79,7 +72,8 @@ function TestCase(frame_index, step_count, expected_final_state, action) {
if (frame_index !== undefined) {
context_frame = exec_state.frame(frame_index);
}
- exec_state.prepareStep(action, step_count, context_frame);
+ exec_state.prepareStep(Debug.StepAction.StepNext,
+ step_count, context_frame);
listener_state = 1;
} else if (listener_state == 1) {
state_snapshot = String(state);
@@ -109,61 +103,30 @@ function TestCase(frame_index, step_count, expected_final_state, action) {
assertEquals(expected_final_state, state_snapshot);
}
-function TestCase2(frame_index, step_count, expected_final_state) {
- return TestCase(frame_index, step_count, expected_final_state,
- Debug.StepAction.StepIn);
-}
// Warm-up -- make sure all is compiled and ready for breakpoint.
h();
-// Stepping over on the default (top) frame.
-TestCase(undefined, 0, "0,0,-1,-1");
-TestCase(undefined, 1, "0,0,-1,-1");
-TestCase(undefined, 2, "0,0,0,-1");
-TestCase(undefined, 5, "0,0,1,-1");
-TestCase(undefined, 8, "0,0,3,-1");
-
-// Stepping over on the frame #0 (should be exactly the same as above).
-TestCase(0, 0, "0,0,-1,-1");
-TestCase(0, 1, "0,0,-1,-1");
-TestCase(0, 2, "0,0,0,-1");
-TestCase(0, 5, "0,0,1,-1");
-TestCase(0, 8, "0,0,3,-1");
-
-// Stepping over on the frame #1.
-TestCase(1, 0, "0,0,5,5");
-TestCase(1, 3, "0,1,5,5");
-TestCase(1, 8, "0,4,5,5");
-
-// Stepping over on the frame #2.
-TestCase(2, 3, "1,5,5,5");
-TestCase(2, 8, "4,5,5,5");
-
-// Stepping into on the default (top) frame.
-TestCase2(undefined, 0, "0,0,-1,-1");
-TestCase2(undefined, 1, "0,0,-1,-1");
-TestCase2(undefined, 2, "0,0,0,-1");
-TestCase2(undefined, 5, "0,0,1,-1");
-TestCase2(undefined, 8, "0,0,3,-1");
-
-// Stepping into on the frame #0 (should be exactly the same as above).
-TestCase2(0, 0, "0,0,-1,-1");
-TestCase2(0, 1, "0,0,-1,-1");
-TestCase2(0, 2, "0,0,0,-1");
-TestCase2(0, 5, "0,0,1,-1");
-TestCase2(0, 8, "0,0,3,-1");
-
-// Stepping into on the frame #1.
-TestCase2(1, 0, "0,0,5,-1");
-TestCase2(1, 3, "0,0,5,0");
-TestCase2(1, 8, "0,0,5,2");
-TestCase2(1, 9, "0,0,5,3");
-
-// Stepping into on the frame #2.
-TestCase2(2, 0, "0,5,5,5");
-TestCase2(2, 3, "1,5,5,5");
-TestCase2(2, 4, "1,0,5,5");
-TestCase2(2, 8, "1,0,0,5");
-TestCase2(2, 9, "1,0,1,5");
+// 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,5");
+TestCase(1, 3, "0,1,5");
+TestCase(1, 8, "0,4,5");
+
+// Stepping in the frame #2.
+TestCase(2, 3, "1,5,5");
+TestCase(2, 8, "4,5,5");
« no previous file with comments | « src/runtime.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698