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

Unified Diff: test/inspector/cpu-profiler/record-cpu-profile.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/cpu-profiler/record-cpu-profile.js
diff --git a/test/inspector/cpu-profiler/record-cpu-profile.js b/test/inspector/cpu-profiler/record-cpu-profile.js
index 4bbdafc06e633dd328b4fb00ee2ede5e098c6490..3ce16fcfb5151738297671fb83565bc5bbd34884 100644
--- a/test/inspector/cpu-profiler/record-cpu-profile.js
+++ b/test/inspector/cpu-profiler/record-cpu-profile.js
@@ -4,41 +4,41 @@
print("Test that profiler is able to record a profile. Also it tests that profiler returns an error when it unable to find the profile.");
-InspectorTest.sendCommand("Profiler.enable", {});
-InspectorTest.sendCommand("Profiler.start", {}, didStartFrontendProfile);
+Protocol.Profiler.enable();
+Protocol.Profiler.start().then(didStartFrontendProfile);
function didStartFrontendProfile(messageObject)
{
if (!InspectorTest.expectedSuccess("startFrontendProfile", messageObject))
return;
- InspectorTest.sendCommand("Runtime.evaluate", {expression: "console.profile('Profile 1');"}, didStartConsoleProfile);
+ Protocol.Runtime.evaluate({expression: "console.profile('Profile 1');"}).then(didStartConsoleProfile);
}
function didStartConsoleProfile(messageObject)
{
if (!InspectorTest.expectedSuccess("startConsoleProfile", messageObject))
return;
- InspectorTest.sendCommand("Runtime.evaluate", {expression: "console.profileEnd('Profile 1');"}, didStopConsoleProfile);
+ Protocol.Runtime.evaluate({expression: "console.profileEnd('Profile 1');"}).then(didStopConsoleProfile);
}
function didStopConsoleProfile(messageObject)
{
if (!InspectorTest.expectedSuccess("stopConsoleProfile", messageObject))
return;
- InspectorTest.sendCommand("Profiler.stop", {}, didStopFrontendProfile);
+ Protocol.Profiler.stop().then(didStopFrontendProfile);
}
function didStopFrontendProfile(messageObject)
{
if (!InspectorTest.expectedSuccess("stoppedFrontendProfile", messageObject))
return;
- InspectorTest.sendCommand("Profiler.start", {}, didStartFrontendProfile2);
+ Protocol.Profiler.start().then(didStartFrontendProfile2);
}
function didStartFrontendProfile2(messageObject)
{
if (!InspectorTest.expectedSuccess("startFrontendProfileSecondTime", messageObject))
return;
- InspectorTest.sendCommand("Profiler.stop", {}, didStopFrontendProfile2);
+ Protocol.Profiler.stop().then(didStopFrontendProfile2);
}
function didStopFrontendProfile2(messageObject)
« no previous file with comments | « test/inspector/cpu-profiler/enable-disable.js ('k') | test/inspector/cpu-profiler/stop-without-preceeding-start.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698