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

Unified Diff: test/debugger/debug/debug-stepin-foreach.js

Issue 2466273005: [debugger] Further stepping support in test wrapper (Closed)
Patch Set: Move more tests Created 4 years, 1 month 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/debugger/debug/debug-stepin-constructor.js ('k') | test/debugger/debug/debug-stepin-function-call.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/debugger/debug/debug-stepin-foreach.js
diff --git a/test/mjsunit/debug-stepin-foreach.js b/test/debugger/debug/debug-stepin-foreach.js
similarity index 79%
rename from test/mjsunit/debug-stepin-foreach.js
rename to test/debugger/debug/debug-stepin-foreach.js
index 69ce3efab75763b84896dbfc9767bb4d0bfa076f..94a6341368dbc92a53802f709e43c4a35bbf2561 100644
--- a/test/mjsunit/debug-stepin-foreach.js
+++ b/test/debugger/debug/debug-stepin-foreach.js
@@ -2,24 +2,17 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-// Flags: --expose-debug-as debug
// Tests stepping into through Array.prototype.forEach callbacks.
Debug = debug.Debug
var exception = null;
var break_count = 0;
-var expected_breaks = -1;
+var expected_breaks = 11;
function listener(event, exec_state, event_data, data) {
try {
if (event == Debug.DebugEvent.Break) {
assertTrue(exec_state.frameCount() != 0, "FAIL: Empty stack trace");
- if (!break_count) {
- // Count number of expected breakpoints in this source file.
- var source_text = exec_state.frame(0).func().script().source();
- expected_breaks = source_text.match(/\/\/\s*Break\s+\d+\./g).length;
- print("Expected breaks: " + expected_breaks);
- }
var source = exec_state.frame(0).sourceLineText();
print("paused at: " + source);
assertTrue(source.indexOf("// Break " + break_count + ".") > 0,
« no previous file with comments | « test/debugger/debug/debug-stepin-constructor.js ('k') | test/debugger/debug/debug-stepin-function-call.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698