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

Side by Side Diff: test/mjsunit/debug-allscopes-on-debugger.js

Issue 1032353002: Make sure debugger is ready for breakpoins when we process 'debugger' statement. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: TODO to clean-up threading issues added Created 5 years, 9 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 unified diff | Download patch
« no previous file with comments | « src/debug.cc ('k') | test/mjsunit/es6/regress/regress-468661.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 the V8 project authors. All rights reserved. 1 // Copyright 2015 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Flags: --expose-debug-as debug 5 // Flags: --expose-debug-as debug
6 6
7 Debug = debug.Debug 7 Debug = debug.Debug
8 var exception = null; 8 var exception = null;
9 var break_count = 0; 9 var break_count = 0;
10 10
11 var expected_values =
12 [ReferenceError, ReferenceError, 0, 0, 0, 0, 0, 1, ReferenceError, ReferenceEr ror];
13
14 function listener(event, exec_state, event_data, data) { 11 function listener(event, exec_state, event_data, data) {
15 try { 12 try {
16 if (event == Debug.DebugEvent.Break) { 13 if (event == Debug.DebugEvent.Break) {
17 assertTrue(exec_state.frameCount() != 0, "FAIL: Empty stack trace"); 14 assertTrue(exec_state.frameCount() != 0, "FAIL: Empty stack trace");
18 // Count number of expected breakpoints in this source file. 15 // Count number of expected breakpoints in this source file.
19 if (!break_count) { 16 if (!break_count) {
20 var source_text = exec_state.frame(0).func().script().source(); 17 var source_text = exec_state.frame(0).func().script().source();
21 expected_breaks = source_text.match(/\/\/\s*Break\s+\d+\./g).length; 18 expected_breaks = source_text.match(/\/\/\s*Break\s+\d+\./g).length;
22 print("Expected breaks: " + expected_breaks); 19 print("Expected breaks: " + expected_breaks);
23 } 20 }
24 var frameMirror = exec_state.frame(0); 21 var frameMirror = exec_state.frame(0);
25 22
26 var v = null;; 23 frameMirror.allScopes();
27 try {
28 v = frameMirror.evaluate('i').value();
29 } catch(e) {
30 v = e;
31 }
32
33 var source = frameMirror.sourceLineText(); 24 var source = frameMirror.sourceLineText();
34 print("paused at: " + source); 25 print("paused at: " + source);
35 assertTrue(source.indexOf("// Break " + break_count + ".") > 0, 26 assertTrue(source.indexOf("// Break " + break_count + ".") > 0,
36 "Unexpected pause at: " + source + "\n" + 27 "Unexpected pause at: " + source + "\n" +
37 "Expected: // Break " + break_count + "."); 28 "Expected: // Break " + break_count + ".");
38 if (expected_values[break_count] === ReferenceError) {
39 assertTrue(v instanceof ReferenceError);
40 } else {
41 assertSame(expected_values[break_count], v);
42 }
43 ++break_count; 29 ++break_count;
44 30
45 if (break_count !== expected_breaks) { 31 if (break_count !== expected_breaks) {
46 exec_state.prepareStep(Debug.StepAction.StepIn, 1); 32 exec_state.prepareStep(Debug.StepAction.StepIn, 1);
47 print("Next step prepared"); 33 print("Next step prepared");
48 } 34 }
49 } 35 }
50 } catch(e) { 36 } catch(e) {
51 exception = e; 37 exception = e;
52 print(e, e.stack); 38 print(e, e.stack);
53 } 39 }
54 }; 40 };
55 41
56 Debug.setListener(listener); 42 Debug.setListener(listener);
57 43
58 var sum = 0; 44 var sum = 0;
59 (function (){ 45 (function (){
60 'use strict'; 46 'use strict';
61 47
62 debugger; // Break 0. 48 debugger; // Break 0.
49 var i = 0; // Break 1.
50 i++; // Break 2.
51 i++; // Break 3.
52 return i; // Break 4.
53 }()); // Break 5.
63 54
64 for (let i=0; // Break 1. 55 assertNull(exception); // Break 6.
65 i < 1; // Break 2. // Break 3. // Break 6. // Break 7.
66 i++) {
67 let key = i; // Break 4.
68 sum += key; // Break 5.
69 }
70 }()); // Break 8.
71
72 assertNull(exception); // Break 9.
73 assertEquals(expected_breaks, break_count); 56 assertEquals(expected_breaks, break_count);
74 57
75 Debug.setListener(null); 58 Debug.setListener(null);
OLDNEW
« no previous file with comments | « src/debug.cc ('k') | test/mjsunit/es6/regress/regress-468661.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698