Skip to content

V
V8
Commit 9db139e1 authored by yangguo's avatar yangguo Committed by Commit bot
Browse files
Options

[inspector] fix coverage test flakiness. 

TBR=kozyatinskiy@chromium.org
BUG=chromium:695631

Review-Url: https://codereview.chromium.org/2718543005
Cr-Commit-Position: refs/heads/master@{#43406}
parent d3cc730b
Hide whitespace changes
Inline Side-by-side
Showing with 11 additions and 6 deletions
test/inspector/runtime/coverage.js
View file @ 9db139e1
...@@ -23,6 +23,11 @@ function ClearAndGC() { ...@@ -23,6 +23,11 @@ function ClearAndGC() {
.then(() => Protocol.HeapProfiler.disable()); .then(() => Protocol.HeapProfiler.disable());
} }
function LogSorted(message) {
message.result.result.sort((a, b) => parseInt(a.scriptId) - parseInt(b.scriptId));
return InspectorTest.logMessage(message);
}
InspectorTest.runTestSuite([ InspectorTest.runTestSuite([
function testPreciseCoverage(next) function testPreciseCoverage(next)
{ {
...@@ -33,9 +38,9 @@ InspectorTest.runTestSuite([ ...@@ -33,9 +38,9 @@ InspectorTest.runTestSuite([
.then(ClearAndGC) .then(ClearAndGC)
.then(InspectorTest.logMessage) .then(InspectorTest.logMessage)
.then(Protocol.Runtime.takePreciseCoverage) .then(Protocol.Runtime.takePreciseCoverage)
.then(InspectorTest.logMessage) .then(LogSorted)
.then(Protocol.Runtime.takePreciseCoverage) .then(Protocol.Runtime.takePreciseCoverage)
.then(InspectorTest.logMessage) .then(LogSorted)
.then(ClearAndGC) .then(ClearAndGC)
.then(Protocol.Runtime.stopPreciseCoverage) .then(Protocol.Runtime.stopPreciseCoverage)
.then(Protocol.Runtime.disable) .then(Protocol.Runtime.disable)
...@@ -62,9 +67,9 @@ InspectorTest.runTestSuite([ ...@@ -62,9 +67,9 @@ InspectorTest.runTestSuite([
.then(InspectorTest.logMessage) .then(InspectorTest.logMessage)
.then(ClearAndGC) .then(ClearAndGC)
.then(Protocol.Runtime.getBestEffortCoverage) .then(Protocol.Runtime.getBestEffortCoverage)
.then(InspectorTest.logMessage) .then(LogSorted)
.then(Protocol.Runtime.getBestEffortCoverage) .then(Protocol.Runtime.getBestEffortCoverage)
.then(InspectorTest.logMessage) .then(LogSorted)
.then(ClearAndGC) .then(ClearAndGC)
.then(Protocol.Runtime.disable) .then(Protocol.Runtime.disable)
.then(next); .then(next);
...@@ -78,9 +83,9 @@ InspectorTest.runTestSuite([ ...@@ -78,9 +83,9 @@ InspectorTest.runTestSuite([
.then(InspectorTest.logMessage) .then(InspectorTest.logMessage)
.then(ClearAndGC) .then(ClearAndGC)
.then(Protocol.Runtime.getBestEffortCoverage) .then(Protocol.Runtime.getBestEffortCoverage)
.then(InspectorTest.logMessage) .then(LogSorted)
.then(Protocol.Runtime.getBestEffortCoverage) .then(Protocol.Runtime.getBestEffortCoverage)
.then(InspectorTest.logMessage) .then(LogSorted)
.then(ClearAndGC) .then(ClearAndGC)
.then(Protocol.Runtime.stopPreciseCoverage) .then(Protocol.Runtime.stopPreciseCoverage)
.then(Protocol.Runtime.disable) .then(Protocol.Runtime.disable)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment