DevTools: merge UISourceCode's parentPath, name, originURL and uri.

third_party/WebKit/Source/devtools/front_end/bindings/FileSystemWorkspaceBinding.js

 /*
  * Copyright (C) 2013 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 48 matching lines @@
 WebInspector.FileSystemWorkspaceBinding._imageExtensions = WebInspector.IsolatedFileSystem.ImageExtensions;
 
 WebInspector.FileSystemWorkspaceBinding._lastRequestId = 0;
 
 /**
  * @param {string} fileSystemPath
  * @return {string}
  */
 WebInspector.FileSystemWorkspaceBinding.projectId = function(fileSystemPath)
 {
-    return "filesystem:" + fileSystemPath;
+    return "file://" + fileSystemPath;
 }
 
 /**
+ * @param {!WebInspector.UISourceCode} uiSourceCode
+ * @return {!Array<string>}
+ */
+WebInspector.FileSystemWorkspaceBinding.relativePath = function(uiSourceCode)
+{
+    var baseURL = /** @type {!WebInspector.FileSystemWorkspaceBinding.FileSystem}*/(uiSourceCode.project())._fileSystemBaseURL;

[dgozman, 2015/12/16 07:58:30]

assert project type?

[pfeldman, 2015/12/16 18:02:18]

It crashes otherwise.

+    return uiSourceCode.path().substring(baseURL.length).split("/");
+}
+
+/**
  * @param {string} extension
  * @return {!WebInspector.ResourceType}
  */
 WebInspector.FileSystemWorkspaceBinding._contentTypeForExtension = function(extension)
 {
     if (WebInspector.FileSystemWorkspaceBinding._styleSheetExtensions.has(extension))
         return WebInspector.resourceTypes.Stylesheet;
     if (WebInspector.FileSystemWorkspaceBinding._documentExtensions.has(extension))
         return WebInspector.resourceTypes.Document;
     if (WebInspector.FileSystemWorkspaceBinding._imageExtensions.has(extension))
@@ skipping 46 matching lines @@
             }
         }
     },
 
     /**
      * @param {string} projectId
      * @return {string}
      */
     fileSystemPath: function(projectId)
     {
-        var fileSystemPath = projectId.substr("filesystem:".length);
-        var normalizedPath = WebInspector.IsolatedFileSystem.normalizePath(fileSystemPath);
-        return projectId.substr("filesystem:".length);
+        return projectId.substr("file://".length);
     },
 
     /**
      * @return {number}
      */
     _nextId: function()
     {
         return ++WebInspector.FileSystemWorkspaceBinding._lastRequestId;
     },
 
@@ skipping 101 matching lines @@
  * @implements {WebInspector.Project}
  * @param {!WebInspector.FileSystemWorkspaceBinding} fileSystemWorkspaceBinding
  * @param {!WebInspector.IsolatedFileSystem} isolatedFileSystem
  * @param {!WebInspector.Workspace} workspace
  */
 WebInspector.FileSystemWorkspaceBinding.FileSystem = function(fileSystemWorkspaceBinding, isolatedFileSystem, workspace)
 {
     this._fileSystemWorkspaceBinding = fileSystemWorkspaceBinding;
     this._fileSystem = isolatedFileSystem;
     this._fileSystemBaseURL = "file://" + this._fileSystem.normalizedPath() + "/";
+    this._fileSystemPath = this._fileSystem.path();
 
-    var id = WebInspector.FileSystemWorkspaceBinding.projectId(this._fileSystem.path());
+    var id = WebInspector.FileSystemWorkspaceBinding.projectId(this._fileSystemPath);
     console.assert(!workspace.project(id));
 
-    var url = "filesystem:" + this._fileSystem.normalizedPath();
     var normalizedPath = isolatedFileSystem.normalizedPath();
     var displayName = normalizedPath.substr(normalizedPath.lastIndexOf("/") + 1);
 
-    WebInspector.ProjectStore.call(this, workspace, id, WebInspector.projectTypes.FileSystem, url, displayName);
+    WebInspector.ProjectStore.call(this, workspace, id, WebInspector.projectTypes.FileSystem, displayName);
 
     workspace.addProject(this);
     this.populate();
 }
 
 WebInspector.FileSystemWorkspaceBinding.FileSystem.prototype = {
     /**
      * @return {string}
      */
     fileSystemPath: function()
     {
-        return this._fileSystem.path();
+        return this._fileSystemPath;
     },
 
     /**
      * @param {!WebInspector.UISourceCode} uiSourceCode
      * @return {string}
      */
     _filePathForUISourceCode: function(uiSourceCode)
     {
-        return "/" + uiSourceCode.path();
+        return uiSourceCode.path().substring(("file:// " + this._fileSystemPath).length);
     },
 
     /**
      * @override
      * @param {!WebInspector.UISourceCode} uiSourceCode
      * @param {function(?string)} callback
      */
     requestFileContent: function(uiSourceCode, callback)
     {
         var filePath = this._filePathForUISourceCode(uiSourceCode);
@@ skipping 153 matching lines @@
         }
     },
 
     /**
      * @param {string} query
      * @param {!WebInspector.Progress} progress
      * @param {function(!Array.<string>)} callback
      */
     _searchInPath: function(query, progress, callback)
     {
-        var requestId = this._fileSystemWorkspaceBinding.registerCallback(innerCallback.bind(this));
+        var requestId = this._fileSystemWorkspaceBinding.registerCallback(innerCallback);
         InspectorFrontendHost.searchInPath(requestId, this._fileSystem.path(), query);
 
         /**
          * @param {!Array.<string>} files
-         * @this {WebInspector.FileSystemWorkspaceBinding.FileSystem}
          */
         function innerCallback(files)
         {
             /**
              * @param {string} fullPath
-             * @this {WebInspector.FileSystemWorkspaceBinding.FileSystem}
+             * @return {string}
              */
             function trimAndNormalizeFileSystemPath(fullPath)
             {
-                var trimmedPath = fullPath.substr(this._fileSystem.path().length + 1);
+                fullPath = "file://" + fullPath;
                 if (WebInspector.isWin())
-                    trimmedPath = trimmedPath.replace(/\\/g, "/");
-                return trimmedPath;
+                    fullPath = fullPath.replace(/\\/g, "/");
+                return fullPath;
             }
 
-            files = files.map(trimAndNormalizeFileSystemPath.bind(this));
+            files = files.map(trimAndNormalizeFileSystemPath);
             progress.worked(1);
             callback(files);
         }
     },
 
     /**
      * @override
      * @param {!WebInspector.Progress} progress
      */
     indexContent: function(progress)
@@ skipping 29 matching lines @@
     {
         this._fileSystem.requestFilesRecursive(path, this._addFile.bind(this), callback);
     },
 
     /**
      * @override
      * @param {string} path
      */
     excludeFolder: function(path)
     {
-        this._fileSystem.addExcludedFolder(path);
+        var relativeFolder = path.substring("file://".length + this._fileSystemPath.length);
+        this._fileSystem.addExcludedFolder(relativeFolder);
+
         var uiSourceCodes = this.uiSourceCodes().slice();
         for (var i = 0; i < uiSourceCodes.length; ++i) {
             var uiSourceCode = uiSourceCodes[i];
-            if (uiSourceCode.path().startsWith(path.substr(1)))
+            if (uiSourceCode.path().startsWith(path))
                 this.removeUISourceCode(uiSourceCode.path());
         }
     },
 
     /**
      * @override
      * @param {string} path
      * @param {?string} name
      * @param {string} content
      * @param {function(?WebInspector.UISourceCode)} callback
@@ skipping 50 matching lines @@
 
     /**
      * @param {string} filePath
      * @return {!WebInspector.UISourceCode}
      */
     _addFile: function(filePath)
     {
         if (!filePath)
             console.assert(false);
 
-        var slash = filePath.lastIndexOf("/");
-        var parentPath = filePath.substring(0, slash);
-        var name = filePath.substring(slash + 1);
-
-        var extension = this._extensionForPath(name);
+        var extension = this._extensionForPath(filePath);
         var contentType = WebInspector.FileSystemWorkspaceBinding._contentTypeForExtension(extension);
 
-        var uiSourceCode = this.createUISourceCode(parentPath, name, this._fileSystemBaseURL + filePath, contentType);
+        var uiSourceCode = this.createUISourceCode(this._fileSystemBaseURL + filePath, contentType);
         this.addUISourceCode(uiSourceCode);
         return uiSourceCode;
     },
 
     /**
      * @param {string} path
      */
     _fileChanged: function(path)
     {
         var uiSourceCode = this.uiSourceCode(path);
         if (!uiSourceCode) {
             this._addFile(path);
             return;
         }
         uiSourceCode.checkContentUpdated();
     },
 
     dispose: function()
     {
         this.removeProject();
     },
 
     __proto__: WebInspector.ProjectStore.prototype
 }