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

Unified Diff: test/inspector/cpu-profiler/coverage.js

Issue 2715833003: [inspector] move coverage related methods to profiler (Closed)
Patch Set: added missing test Created 3 years, 10 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/coverage.js
diff --git a/test/inspector/runtime/coverage.js b/test/inspector/cpu-profiler/coverage.js
similarity index 74%
rename from test/inspector/runtime/coverage.js
rename to test/inspector/cpu-profiler/coverage.js
index dff7eca3893aba64916735700fa68dbb4e96efda..c5abd4e517f76727b24c80adb17263d819925b37 100644
--- a/test/inspector/runtime/coverage.js
+++ b/test/inspector/cpu-profiler/coverage.js
@@ -14,7 +14,7 @@ function fib(x) {
fib(5);
`;
-print("Test collecting code coverage data with Runtime.collectCoverage.");
+print("Test collecting code coverage data with Profiler.collectCoverage.");
function ClearAndGC() {
return Protocol.Runtime.evaluate({ expression: "fib = null;" })
@@ -27,30 +27,34 @@ InspectorTest.runTestSuite([
function testPreciseCoverage(next)
{
Protocol.Runtime.enable()
- .then(Protocol.Runtime.startPreciseCoverage)
+ .then(Protocol.Profiler.enable)
+ .then(Protocol.Profiler.startPreciseCoverage)
.then(() => Protocol.Runtime.compileScript({ expression: source, sourceURL: "1", persistScript: true }))
.then((result) => Protocol.Runtime.runScript({ scriptId: result.result.scriptId }))
.then(ClearAndGC)
.then(InspectorTest.logMessage)
- .then(Protocol.Runtime.takePreciseCoverage)
+ .then(Protocol.Profiler.takePreciseCoverage)
.then(InspectorTest.logMessage)
- .then(Protocol.Runtime.takePreciseCoverage)
+ .then(Protocol.Profiler.takePreciseCoverage)
.then(InspectorTest.logMessage)
.then(ClearAndGC)
- .then(Protocol.Runtime.stopPreciseCoverage)
+ .then(Protocol.Profiler.stopPreciseCoverage)
+ .then(Protocol.Profiler.disable)
.then(Protocol.Runtime.disable)
.then(next);
},
function testPreciseCoverageFail(next)
{
Protocol.Runtime.enable()
+ .then(Protocol.Profiler.enable)
.then(() => Protocol.Runtime.compileScript({ expression: source, sourceURL: "2", persistScript: true }))
.then((result) => Protocol.Runtime.runScript({ scriptId: result.result.scriptId }))
.then(InspectorTest.logMessage)
.then(ClearAndGC)
- .then(Protocol.Runtime.takePreciseCoverage)
+ .then(Protocol.Profiler.takePreciseCoverage)
.then(InspectorTest.logMessage)
.then(ClearAndGC)
+ .then(Protocol.Profiler.disable)
.then(Protocol.Runtime.disable)
.then(next);
},
@@ -61,9 +65,9 @@ InspectorTest.runTestSuite([
.then((result) => Protocol.Runtime.runScript({ scriptId: result.result.scriptId }))
.then(InspectorTest.logMessage)
.then(ClearAndGC)
- .then(Protocol.Runtime.getBestEffortCoverage)
+ .then(Protocol.Profiler.getBestEffortCoverage)
.then(InspectorTest.logMessage)
- .then(Protocol.Runtime.getBestEffortCoverage)
+ .then(Protocol.Profiler.getBestEffortCoverage)
.then(InspectorTest.logMessage)
.then(ClearAndGC)
.then(Protocol.Runtime.disable)
@@ -72,17 +76,19 @@ InspectorTest.runTestSuite([
function testBestEffortCoveragePrecise(next)
{
Protocol.Runtime.enable()
- .then(Protocol.Runtime.startPreciseCoverage)
+ .then(Protocol.Profiler.enable)
+ .then(Protocol.Profiler.startPreciseCoverage)
.then(() => Protocol.Runtime.compileScript({ expression: source, sourceURL: "4", persistScript: true }))
.then((result) => Protocol.Runtime.runScript({ scriptId: result.result.scriptId }))
.then(InspectorTest.logMessage)
.then(ClearAndGC)
- .then(Protocol.Runtime.getBestEffortCoverage)
+ .then(Protocol.Profiler.getBestEffortCoverage)
.then(InspectorTest.logMessage)
- .then(Protocol.Runtime.getBestEffortCoverage)
+ .then(Protocol.Profiler.getBestEffortCoverage)
.then(InspectorTest.logMessage)
.then(ClearAndGC)
- .then(Protocol.Runtime.stopPreciseCoverage)
+ .then(Protocol.Profiler.stopPreciseCoverage)
+ .then(Protocol.Profiler.disable)
.then(Protocol.Runtime.disable)
.then(next);
},

Powered by Google App Engine
This is Rietveld 408576698