Devtools: Pretty print fix for CSS coverage decorations.

third_party/WebKit/Source/devtools/front_end/sources/InplaceFormatterEditorAction.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 /**
  * @implements {Sources.SourcesView.EditorAction}
  * @unrestricted
  */
 Sources.InplaceFormatterEditorAction = class {
   /**
    * @param {!Common.Event} event
@@ skipping 81 matching lines @@
     function innerCallback(formattedContent, formatterMapping) {
       if (uiSourceCode.workingCopy() === formattedContent)
         return;
       var sourceFrame = this._sourcesView.viewForFile(uiSourceCode);
       var start = [0, 0];
       if (sourceFrame) {
         var selection = sourceFrame.selection();
         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 formattedStartLine = sourceMapping.originalToFormatted(decoration.range().startLine,

[caseq, 2016/11/17 22:13:19]

nit: var range = decoration.range() for brevity.

+                                                                    decoration.range().startColumn)[0];
+      var formattedStartColumn = sourceMapping.originalToFormatted(decoration.range().startLine,

[caseq, 2016/11/17 22:13:19]

why are we calling it twice with the same arguments?

+                                                                    decoration.range().startColumn)[1];
+
+      var formattedEndLine = sourceMapping.originalToFormatted(decoration.range().endLine,
+                                                                decoration.range().endColumn)[0];

[caseq, 2016/11/17 22:13:19]

let's just indent +4 spaces from the start of previous line on expression
continuation, it's difficult to read otherwise.

+      var formattedEndColumn = sourceMapping.originalToFormatted(decoration.range().endLine,
+                                                                  decoration.range().endColumn)[1];
+
+      uiSourceCode.addDecoration(new Common.TextRange(formattedStartLine, formattedStartColumn,
+                                                  formattedEndLine, formattedEndColumn),
+                                  /** @type {string} */ (decoration.type()),
+                                  decoration.data());
+    }
+  }
 };