third_party/WebKit/LayoutTests/inspector/coverage/coverage-test.js - Issue 2865573003: DevTools: support live coverage

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: third_party/WebKit/LayoutTests/inspector/coverage/coverage-test.js

Issue 2865573003: DevTools: support live coverage (Closed)

Patch Set: fixed accidentally creating too many promises for one resource 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:

Index: third_party/WebKit/LayoutTests/inspector/coverage/coverage-test.js

diff --git a/third_party/WebKit/LayoutTests/inspector/coverage/coverage-test.js b/third_party/WebKit/LayoutTests/inspector/coverage/coverage-test.js

index 1277fe3bda9eba6c76c792fa3ca0345dfd778f92..a3d9aba02d1c8b8c11adabb9fb72244ed8a5ddba 100644

--- a/third_party/WebKit/LayoutTests/inspector/coverage/coverage-test.js

+++ b/third_party/WebKit/LayoutTests/inspector/coverage/coverage-test.js

@@ -18,7 +18,7 @@ InspectorTest.stopCoverage = function()

InspectorTest.sourceDecorated = async function(source)

{

await UI.inspectorView.showPanel("sources");

- var decoratePromise = InspectorTest.addSnifferPromise(Coverage.CoverageView.LineDecorator.prototype, "decorate");

+ var decoratePromise = InspectorTest.addSnifferPromise(Coverage.CoverageView.LineDecorator.prototype, "_innerDecorate");

var sourceFrame = await new Promise(fulfill => InspectorTest.showScriptSource(source, fulfill));

await decoratePromise;

return sourceFrame;

@@ -43,7 +43,7 @@ InspectorTest.findCoverageNodeForURL = function(url)

InspectorTest.dumpDecorationsInSourceFrame = function(sourceFrame)

{

- var markerMap = new Map([['used', '+'], ['unused', '-'], ['mixed', '*']]);

+ var markerMap = new Map([['used', '+'], ['unused', '-']]);

var codeMirror = sourceFrame.textEditor.codeMirror();

for (var line = 0; line < codeMirror.lineCount(); ++line) {