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

Unified Diff: test/inspector/debugger/framework-stepping.js

Issue 2891213002: [inspector] Refactor protocol-test.js (Closed)
Patch Set: comments addressed Created 3 years, 7 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
Index: test/inspector/debugger/framework-stepping.js
diff --git a/test/inspector/debugger/framework-stepping.js b/test/inspector/debugger/framework-stepping.js
index 5135d0f81da5e2eff0f46cad6d3edbaf496b1c6c..f91c06ba52d1ac3e02f69566357c77927dd60919 100644
--- a/test/inspector/debugger/framework-stepping.js
+++ b/test/inspector/debugger/framework-stepping.js
@@ -2,9 +2,9 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-InspectorTest.log('Checks stepping with blackboxed frames on stack');
+let {session, contextGroup, Protocol} = InspectorTest.start('Checks stepping with blackboxed frames on stack');
-InspectorTest.addScript(
+contextGroup.addScript(
`
function frameworkCall(funcs) {
for (var f of funcs) f();
@@ -17,7 +17,7 @@ function frameworkBreakAndCall(funcs) {
//# sourceURL=framework.js`,
8, 4);
-InspectorTest.addScript(
+contextGroup.addScript(
`
function userFoo() {
return 1;
@@ -37,7 +37,7 @@ function testStepFromFramework() {
//# sourceURL=user.js`,
21, 4);
-InspectorTest.setupScriptMap();
+session.setupScriptMap();
Protocol.Debugger.enable()
.then(
@@ -47,7 +47,7 @@ Protocol.Debugger.enable()
var testSuite = [
function testStepIntoFromUser(next) {
- InspectorTest.contextGroup.schedulePauseOnNextStatement('', '');
+ contextGroup.schedulePauseOnNextStatement('', '');
test('testStepFromUser()', [
'print', // before testStepFromUser call
'stepInto', 'stepInto', 'print', // userFoo
@@ -57,7 +57,7 @@ var testSuite = [
},
function testStepOverFromUser(next) {
- InspectorTest.contextGroup.schedulePauseOnNextStatement('', '');
+ contextGroup.schedulePauseOnNextStatement('', '');
test('testStepFromUser()', [
'print', // before testStepFromUser call
'stepInto', 'stepInto', 'print', // userFoo
@@ -67,7 +67,7 @@ var testSuite = [
},
function testStepOutFromUser(next) {
- InspectorTest.contextGroup.schedulePauseOnNextStatement('', '');
+ contextGroup.schedulePauseOnNextStatement('', '');
test('testStepFromUser()', [
'print', // before testStepFromUser call
'stepInto', 'stepInto', 'print', // userFoo
@@ -101,7 +101,7 @@ function test(entryExpression, actions) {
Protocol.Debugger.onPaused(message => {
var action = actions.shift() || 'resume';
if (action === 'print') {
- InspectorTest.logCallFrames(message.params.callFrames);
+ session.logCallFrames(message.params.callFrames);
InspectorTest.log('');
action = actions.shift() || 'resume';
}
« no previous file with comments | « test/inspector/debugger/framework-precise-ranges.js ('k') | test/inspector/debugger/get-possible-breakpoints.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698