DevTools: remove WI.NetworkMapping.networkURL

third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.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
  * @param {!WebInspector.TargetManager} targetManager
  * @param {!WebInspector.Workspace} workspace
  * @param {!WebInspector.FileSystemWorkspaceBinding} fileSystemWorkspaceBinding
  * @param {!WebInspector.FileSystemMapping} fileSystemMapping
@@ skipping 59 matching lines @@
     /**
      * @param {!WebInspector.Event} event
      */
     _fileSystemRemoved: function(event)
     {
         var fileSystem = /** @type {!WebInspector.IsolatedFileSystem} */ (event.data);
         this._fileSystemMapping.removeFileSystem(fileSystem.path());
     },
 
     /**
-     * @param {!WebInspector.UISourceCode} uiSourceCode
-     * @return {string}
-     */
-    networkURL: function(uiSourceCode)
-    {
-        if (uiSourceCode.project().type() === WebInspector.projectTypes.FileSystem)
-            return "";
-        return uiSourceCode.url();
-    },
-
-    /**
      * @param {!WebInspector.Target} target
      * @param {?WebInspector.ResourceTreeFrame} frame
      * @param {string} url
      * @return {?WebInspector.UISourceCode}
      */
     _networkUISourceCodeForURL: function(target, frame, url)
     {
         return this._workspace.uiSourceCode(WebInspector.NetworkProject.projectId(target, frame, false), url);
     },
 
@@ skipping 49 matching lines @@
     {
         return WebInspector.workspace.uiSourceCodeForURL(url);
     },
 
     /**
      * @param {!WebInspector.UISourceCode} networkUISourceCode
      * @param {!WebInspector.UISourceCode} uiSourceCode
      */
     addMapping: function(networkUISourceCode, uiSourceCode)
     {
-        var url = this.networkURL(networkUISourceCode);
-        var path = uiSourceCode.url();
         var fileSystemPath = WebInspector.FileSystemWorkspaceBinding.fileSystemPath(uiSourceCode.project().id());
-        this._fileSystemMapping.addMappingForResource(url, fileSystemPath, path);
+        this._fileSystemMapping.addMappingForResource(networkUISourceCode.url(), fileSystemPath, uiSourceCode.url());
     },
 
     /**
      * @param {!WebInspector.UISourceCode} uiSourceCode
      */
     removeMapping: function(uiSourceCode)

[dgozman, 2016/09/27 17:48:24]

This could have been a file system one. Please double-check it works.

[lushnikov, 2016/09/28 18:36:02]

Done. (@see SourcesPane.js:792)

     {
-        var networkURL = this.networkURL(uiSourceCode);
-        this._fileSystemMapping.removeMappingForURL(networkURL);
+        this._fileSystemMapping.removeMappingForURL(uiSourceCode.url());
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _revealSourceLine: function(event)
     {
         var url = /** @type {string} */ (event.data["url"]);
         var lineNumber = /** @type {number} */ (event.data["lineNumber"]);
         var columnNumber = /** @type {number} */ (event.data["columnNumber"]);
 
         var uiSourceCode = this.uiSourceCodeForURLForAnyTarget(url);
         if (uiSourceCode) {
             WebInspector.Revealer.reveal(uiSourceCode.uiLocation(lineNumber, columnNumber));
             return;
         }
 
         /**
          * @param {!WebInspector.Event} event
          * @this {WebInspector.NetworkMapping}
          */
         function listener(event)
         {
             var uiSourceCode = /** @type {!WebInspector.UISourceCode} */ (event.data);
-            if (this.networkURL(uiSourceCode) === url) {
+            if (uiSourceCode.url() === url) {
                 WebInspector.Revealer.reveal(uiSourceCode.uiLocation(lineNumber, columnNumber));
                 this._workspace.removeEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, listener, this);
             }
         }
 
         this._workspace.addEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, listener, this);
     },
 
     dispose: function()
     {
         WebInspector.EventTarget.removeEventListeners(this._eventListeners);
     }
 }
 
 /**
  * @type {!WebInspector.NetworkMapping}
  */
 WebInspector.networkMapping;