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

Unified Diff: test/mjsunit/es6/debug-step-into-constructor.js

Issue 2469043003: [debugger] fix blacklisted tests. (Closed)
Patch Set: fix 2318 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 | « no previous file | test/mjsunit/es6/default-parameters-debug.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/es6/debug-step-into-constructor.js
diff --git a/test/mjsunit/es6/debug-step-into-constructor.js b/test/mjsunit/es6/debug-step-into-constructor.js
index 96cdc931592ec717154a356d06621efbb17f3893..0e746386d331a1ffc2e688bfce6be09e059cfbe8 100644
--- a/test/mjsunit/es6/debug-step-into-constructor.js
+++ b/test/mjsunit/es6/debug-step-into-constructor.js
@@ -8,15 +8,19 @@
var Debug = debug.Debug
var done, stepCount;
+var exception = null;
function listener(event, execState, eventData, data) {
- if (event == Debug.DebugEvent.Break) {
+ if (event != Debug.DebugEvent.Break) return;
+ try {
if (!done) {
execState.prepareStep(Debug.StepAction.StepIn);
var s = execState.frame().sourceLineText();
assertTrue(s.indexOf('// ' + stepCount + '.') !== -1);
stepCount++;
}
+ } catch (e) {
+ exception = e;
}
};
@@ -24,14 +28,14 @@ Debug.setListener(listener);
class Base {
- constructor() { // 1.
- var x = 1; // 2.
- var y = 2; // 3.
- done = true; // 4.
+ constructor() {
+ var x = 1; // 1.
+ var y = 2; // 2.
+ done = true; // 3.
}
}
-class Derived extends Base {}
+class Derived extends Base {} // 0.
(function TestBreakPointInConstructor() {
@@ -40,7 +44,7 @@ class Derived extends Base {}
var bp = Debug.setBreakPoint(Base, 0);
new Base();
- assertEquals(1, stepCount);
+ assertEquals(4, stepCount);
Debug.clearBreakPoint(bp);
})();
@@ -52,7 +56,7 @@ class Derived extends Base {}
var bp = Debug.setBreakPoint(Base, 0);
new Derived();
- assertEquals(1, stepCount);
+ assertEquals(4, stepCount);
Debug.clearBreakPoint(bp);
})();
@@ -60,15 +64,15 @@ class Derived extends Base {}
(function TestStepInto() {
done = false;
- stepCount = 0;
+ stepCount = -1;
function f() {
- new Derived(); // 0.
+ new Derived(); // -1.
}
var bp = Debug.setBreakPoint(f, 0);
f();
- assertEquals(1, stepCount);
+ assertEquals(4, stepCount);
Debug.clearBreakPoint(bp);
})();
@@ -76,17 +80,17 @@ class Derived extends Base {}
(function TestExtraIndirection() {
done = false;
- stepCount = 0;
+ stepCount = -2;
- class Derived2 extends Derived {}
+ class Derived2 extends Derived {} // -1.
function f() {
- new Derived2(); // 0.
+ new Derived2(); // -2.
}
var bp = Debug.setBreakPoint(f, 0);
f();
- assertEquals(1, stepCount);
+ assertEquals(4, stepCount);
Debug.clearBreakPoint(bp);
})();
@@ -94,20 +98,21 @@ class Derived extends Base {}
(function TestBoundClass() {
done = false;
- stepCount = 0;
+ stepCount = -1;
var bound = Derived.bind(null);
function f() {
- new bound(); // 0.
+ new bound(); // -1.
}
var bp = Debug.setBreakPoint(f, 0);
f();
- assertEquals(1, stepCount);
+ assertEquals(4, stepCount);
Debug.clearBreakPoint(bp);
})();
Debug.setListener(null);
+assertNull(exception);
« no previous file with comments | « no previous file | test/mjsunit/es6/default-parameters-debug.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698