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

Unified Diff: test/inspector/debugger/doesnt-step-into-injected-script.js

Issue 2390733002: [inspector] Make InspectorTest.sendCommand* private (Closed)
Patch Set: addressed comments Created 4 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
Index: test/inspector/debugger/doesnt-step-into-injected-script.js
diff --git a/test/inspector/debugger/doesnt-step-into-injected-script.js b/test/inspector/debugger/doesnt-step-into-injected-script.js
index e5cd55cb3813b14511a03bb9082db0257a28d10b..4d0d1d1f195cbdabaa3b52074f06144e66bf7b43 100644
--- a/test/inspector/debugger/doesnt-step-into-injected-script.js
+++ b/test/inspector/debugger/doesnt-step-into-injected-script.js
@@ -4,15 +4,15 @@
print("Check that stepInto at then end of the script go to next user script instead InjectedScriptSource.js.");
-InspectorTest.evaluateInPage(
+InspectorTest.addScript(
`function foo()
{
return 239;
}`);
-InspectorTest.sendCommandOrDie("Debugger.enable", {});
-InspectorTest.eventHandler["Debugger.paused"] = debuggerPaused;
-InspectorTest.sendCommandOrDie("Runtime.evaluate", { "expression": "(function boo() { setTimeout(foo, 0); debugger; })()" });
+Protocol.Debugger.enable();
+Protocol.Debugger.onPaused(debuggerPaused);
+Protocol.Runtime.evaluate({ "expression": "(function boo() { setTimeout(foo, 0); debugger; })()" });
var actions = [ "stepInto", "stepInto", "stepInto" ];
function debuggerPaused(result)
@@ -24,9 +24,9 @@ function debuggerPaused(result)
var action = actions.shift();
if (!action) {
- InspectorTest.sendCommandOrDie("Debugger.resume", {}, () => InspectorTest.completeTest());
+ Protocol.Debugger.resume().then(InspectorTest.completeTest);
return;
}
InspectorTest.log("Perform " + action);
- InspectorTest.sendCommandOrDie("Debugger." + action, {});
+ Protocol.Debugger[action]();
}
« no previous file with comments | « test/inspector/debugger/continue-to-location.js ('k') | test/inspector/debugger/scope-skip-variables-with-empty-name.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698