[DevTools] Add sourceMap support for blackboxing

third_party/WebKit/Source/devtools/front_end/bindings/BlackboxSupport.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.
 
 /**
  * @constructor
  */
 WebInspector.BlackboxManager = function()
 {
     WebInspector.targetManager.addModelListener(WebInspector.DebuggerModel, WebInspector.DebuggerModel.Events.ParsedScriptSource, this._parsedScriptSource, this);
     WebInspector.moduleSetting("skipStackFramesPattern").addChangeListener(this._patternChanged.bind(this));
     WebInspector.moduleSetting("skipContentScripts").addChangeListener(this._patternChanged.bind(this));
+
+    /** @type {!Map<string, !WebInspector.SourceMap>} */
+    this._sourceMapForScriptId = new Map();

[dgozman, 2016/02/03 18:24:10]

Ask CompilerScriptMapping instead of keeping a separate map.

[kozy, 2016/02/09 23:10:09]

Done.

 }
 
 WebInspector.BlackboxManager.prototype = {
     /**
      * @param {!WebInspector.Event} event
      */
     _parsedScriptSource: function(event)
     {
         var script = /** @type {!WebInspector.Script} */ (event.data);
+        script.addEventListener(WebInspector.Script.Events.SourceMapLoaded, this._sourceMapLoaded.bind(this, script));
         this._blackboxScriptIfNeeded(script);
     },
 
     _patternChanged: function()
     {
         for (var debuggerModel of WebInspector.DebuggerModel.instances()) {
             for (var scriptId in debuggerModel.scripts)
                 this._blackboxScriptIfNeeded(debuggerModel.scripts[scriptId]);
         }
     },
 
     /**
      * @param {!WebInspector.Script} script
      */
     _blackboxScriptIfNeeded: function(script)
     {
-        if (WebInspector.BlackboxSupport.isBlackboxed(script.sourceURL, script.isContentScript()))
+        if (WebInspector.BlackboxSupport.isBlackboxed(script.sourceURL, script.isContentScript())) {
             script.setBlackboxedRanges([ { line: 0, column: 0 } ]);
-        else
+            return;
+        }
+
+        if (!this._sourceMapForScriptId.has(script.scriptId)) {
             script.setBlackboxedRanges([]);
+            return;
+        }
+
+        var sourceMap = this._sourceMapForScriptId.get(script.scriptId);
+        var mappings = sourceMap.mappings().slice();
+        mappings.sort(mappingComparator);
+
+        if (!mappings.length) {
+            script.setBlackboxedRanges([]);
+            return;
+        }
+
+        var currentBlackboxed = false;
+        var positions = [];
+        if (mappings[0].lineNumber !== 0 || mappings[0].columnNumber !== 0) {
+            positions.push({ line: 0, column: 0});
+            currentBlackboxed = true;
+        }
+        for (var mapping of sourceMap.mappings()) {
+            if (currentBlackboxed !== WebInspector.BlackboxSupport.isBlackboxedURL(mapping.sourceURL)) {

[dgozman, 2016/02/03 18:24:10]

Ain't this slow? Should we cache instead of regexp'ing all the time?

[kozy, 2016/02/09 23:10:09]

Done.

+                positions.push({ line: mapping.lineNumber, column: mapping.columnNumber });
+                currentBlackboxed = !currentBlackboxed;
+            }
+        }
+
+        script.setBlackboxedRanges(positions);
+
+        /**
+         * @param {!WebInspector.SourceMap.Entry} a
+         * @param {!WebInspector.SourceMap.Entry} b
+         */
+        function mappingComparator(a, b)
+        {
+            if (a.lineNumber !== b.lineNumber)
+                return a.lineNumber - b.lineNumber;
+            return a.columnNumber - b.columnNumber;
+        }
+    },
+
+    /**
+     * @param {!WebInspector.Script} script
+     * @param {!WebInspector.Event} event
+     */
+    _sourceMapLoaded: function(script, event)
+    {
+        var sourceMap = /** @type {?WebInspector.SourceMap} */ (event.data);
+        if (!sourceMap)
+            return;
+
+        this._sourceMapForScriptId.set(script.scriptId, sourceMap);
+        this._blackboxScriptIfNeeded(script);
     }
 }
 
 WebInspector.BlackboxSupport = {}
 
 /**
  * @param {string} url
  * @return {string}
  */
 WebInspector.BlackboxSupport._urlToRegExpString = function(url)
@@ skipping 117 matching lines @@
 }
 
 /**
  * @param {function(!WebInspector.Event)} listener
  * @param {!Object=} thisObject
  */
 WebInspector.BlackboxSupport.removeChangeListener = function(listener, thisObject)
 {
     WebInspector.moduleSetting("skipStackFramesPattern").removeChangeListener(listener, thisObject);
 }