Devtools: Pretty print fix for CSS coverage decorations.

third_party/WebKit/Source/devtools/front_end/workspace/UISourceCode.js

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 42 matching lines @@
       this._origin = '';
       this._parentURL = '';
       this._name = url;
     }
 
     this._contentType = contentType;
     /** @type {?function(?string)} */
     this._requestContentCallback = null;
     /** @type {?Promise<?string>} */
     this._requestContentPromise = null;
-    /** @type {!Map<string, !Map<number, !Workspace.UISourceCode.LineMarker>>} */
-    this._lineDecorations = new Map();
+    /** @type {!Map<string, !Map<!Common.TextRange, !Workspace.UISourceCode.LineMarker>>} */

[caseq, 2016/11/15 22:07:05]

I wonder if there's any reason to have map now.

+    this._decorations = new Map();
 
     /** @type {!Array.<!Workspace.Revision>} */
     this.history = [];
 
     /** @type {!Array<!Workspace.UISourceCode.Message>} */
     this._messages = [];
   }
 
   /**
    * @return {!Promise<?Workspace.UISourceCodeMetadata>}
@@ skipping 524 matching lines @@
     for (var message of messages)
       this.dispatchEventToListeners(Workspace.UISourceCode.Events.MessageRemoved, message);
   }
 
   /**
    * @param {number} lineNumber
    * @param {string} type
    * @param {?} data
    */
   addLineDecoration(lineNumber, type, data) {
-    var markers = this._lineDecorations.get(type);
+    this.addDecoration(Common.TextRange.createFromLocation(lineNumber, 0), type, data);
+  }
+
+  /**
+   * @param {!Common.TextRange} range
+   * @param {string} type
+   * @param {?} data
+   */
+  addDecoration(range, type, data) {
+    var markers = this._decorations.get(type);
     if (!markers) {
       markers = new Map();
-      this._lineDecorations.set(type, markers);
+      this._decorations.set(type, markers);
     }
-    var marker = new Workspace.UISourceCode.LineMarker(lineNumber, type, data);
-    markers.set(lineNumber, marker);
+    var marker = new Workspace.UISourceCode.LineMarker(range.startLine, type, data);
+    markers.set(range, marker);
     this.dispatchEventToListeners(Workspace.UISourceCode.Events.LineDecorationAdded, marker);
   }
 
   /**
    * @param {number} lineNumber
    * @param {string} type
    */
   removeLineDecoration(lineNumber, type) {
-    var markers = this._lineDecorations.get(type);
+    this.removeDecoration(Common.TextRange.createFromLocation(lineNumber, 0), type);

[caseq, 2016/11/15 22:07:05]

Does this actually work?

+  }
+
+  /**
+   * @param {!Common.TextRange} range
+   * @param {string} type
+   */
+  removeDecoration(range, type) {
+    var markers = this._decorations.get(type);
     if (!markers)
       return;
-    var marker = markers.get(lineNumber);
+    var marker = markers.get(range);
     if (!marker)
       return;
-    markers.delete(lineNumber);
+    markers.delete(range);
     this.dispatchEventToListeners(Workspace.UISourceCode.Events.LineDecorationRemoved, marker);
     if (!markers.size)
-      this._lineDecorations.delete(type);
+      this._decorations.delete(type);
   }
 
   /**
    * @param {string} type
    */
   removeAllLineDecorations(type) {
-    var markers = this._lineDecorations.get(type);
+    var markers = this._decorations.get(type);
     if (!markers)
       return;
-    this._lineDecorations.delete(type);
+    this._decorations.delete(type);
     markers.forEach(marker => {
       this.dispatchEventToListeners(Workspace.UISourceCode.Events.LineDecorationRemoved, marker);
     });
   }
 
   /**
+   * @return {!Map<string, !Map<!Common.TextRange, !Workspace.UISourceCode.LineMarker>>}
+   */
+  getAllDecorations() {
+    return this._decorations;
+  }
+
+  removeAllDecorations() {
+    this._decorations.clear();
+  }
+
+  /**
    * @param {string} type
-   * @return {?Map<number, !Workspace.UISourceCode.LineMarker>}
+   * @return {?Map<!Common.TextRange, !Workspace.UISourceCode.LineMarker>}
    */
   lineDecorations(type) {
-    return this._lineDecorations.get(type) || null;
+    return this._decorations.get(type) || null;
   }
 };
 
 /** @enum {symbol} */
 Workspace.UISourceCode.Events = {
   WorkingCopyChanged: Symbol('WorkingCopyChanged'),
   WorkingCopyCommitted: Symbol('WorkingCopyCommitted'),
   TitleChanged: Symbol('TitleChanged'),
   SourceMappingChanged: Symbol('SourceMappingChanged'),
   MessageAdded: Symbol('MessageAdded'),
@@ skipping 277 matching lines @@
 Workspace.UISourceCodeMetadata = class {
   /**
    * @param {?Date} modificationTime
    * @param {?number} contentSize
    */
   constructor(modificationTime, contentSize) {
     this.modificationTime = modificationTime;
     this.contentSize = contentSize;
   }
 };