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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sources/InplaceFormatterEditorAction.js

Issue 2492343002: Devtools: Pretty print fix for CSS coverage decorations. (Closed)
Patch Set: Pretty print fix for CSS coverage decorations. Created 4 years, 1 month 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: third_party/WebKit/Source/devtools/front_end/sources/InplaceFormatterEditorAction.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/InplaceFormatterEditorAction.js b/third_party/WebKit/Source/devtools/front_end/sources/InplaceFormatterEditorAction.js
index b703d279eb5cec8055e0a3d25cdda210bc4dd2c6..f2c6c34758a3b767399ab86b1d81c3208543a52a 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/InplaceFormatterEditorAction.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/InplaceFormatterEditorAction.js
@@ -99,7 +99,32 @@ Sources.InplaceFormatterEditorAction = class {
start = formatterMapping.originalToFormatted(selection.startLine, selection.startColumn);
}
uiSourceCode.setWorkingCopy(formattedContent);
+ this._formatDecorations(uiSourceCode, formatterMapping);
+
this._sourcesView.showSourceLocation(uiSourceCode, start[0], start[1]);
}
}
+
+ /**
+ * @param {!Workspace.UISourceCode} uiSourceCode
+ * @param {!Sources.FormatterSourceMapping} sourceMapping
+ */
+ _formatDecorations(uiSourceCode, sourceMapping) {
+ var decorations = uiSourceCode.allDecorations();
+ if (!decorations.length)
+ return;
+
+ uiSourceCode.removeAllDecorations();
+
+ for (var decoration of decorations) {
+ var range = decoration.range();
+ var startLocation = sourceMapping.originalToFormatted(range.startLine, range.startColumn);
+ var endLocation = sourceMapping.originalToFormatted(range.endLine, range.endColumn);
+
+ uiSourceCode.addDecoration(
+ new Common.TextRange(...startLocation, ...endLocation),
+ /** @type {string} */ (decoration.type()),
+ decoration.data());
+ }
+ }
};

Powered by Google App Engine
This is Rietveld 408576698