Title: [143174] trunk
Revision
143174
Author
commit-qu...@webkit.org
Date
2013-02-18 01:30:22 -0800 (Mon, 18 Feb 2013)

Log Message

Unreviewed, rolling out r143100.
http://trac.webkit.org/changeset/143100
https://bugs.webkit.org/show_bug.cgi?id=110088

Breaks file system support in workspace. (Requested by vsevik
on #webkit).

Patch by Sheriff Bot <webkit.review....@gmail.com> on 2013-02-18

Source/WebCore:

* inspector/front-end/ScriptSnippetModel.js:
(WebInspector.ScriptSnippetModel.prototype._addScriptSnippet):
* inspector/front-end/SimpleWorkspaceProvider.js:
(WebInspector.SimpleProjectDelegate):
(WebInspector.SimpleProjectDelegate.prototype.id):
(WebInspector.SimpleProjectDelegate.prototype.displayName):
(WebInspector.SimpleProjectDelegate.prototype.addFile):
(WebInspector.SimpleWorkspaceProvider):
(WebInspector.SimpleWorkspaceProvider.uriForURL):
(WebInspector.SimpleWorkspaceProvider.prototype._innerAddFileForURL):
(WebInspector.SimpleWorkspaceProvider.prototype.removeFile):
(WebInspector.SimpleWorkspaceProvider.prototype.reset):
* inspector/front-end/UISourceCode.js:
(WebInspector.UISourceCode.prototype.uri):
* inspector/front-end/Workspace.js:
(WebInspector.Project.prototype._fileAdded):
(WebInspector.Project.prototype._fileRemoved):
(WebInspector.Project.prototype.uiSourceCodeForURI):
(WebInspector.Workspace.prototype.projectForUISourceCode):

LayoutTests:

* inspector/debugger/live-edit-breakpoints-expected.txt:
* inspector/debugger/live-edit-breakpoints.html:
* inspector/uisourcecode-revisions.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (143173 => 143174)


--- trunk/LayoutTests/ChangeLog	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/LayoutTests/ChangeLog	2013-02-18 09:30:22 UTC (rev 143174)
@@ -1,3 +1,16 @@
+2013-02-18  Sheriff Bot  <webkit.review....@gmail.com>
+
+        Unreviewed, rolling out r143100.
+        http://trac.webkit.org/changeset/143100
+        https://bugs.webkit.org/show_bug.cgi?id=110088
+
+        Breaks file system support in workspace. (Requested by vsevik
+        on #webkit).
+
+        * inspector/debugger/live-edit-breakpoints-expected.txt:
+        * inspector/debugger/live-edit-breakpoints.html:
+        * inspector/uisourcecode-revisions.html:
+
 2013-02-18  Zan Dobersek  <zdober...@igalia.com>
 
         Unreviewed gardening.

Modified: trunk/LayoutTests/inspector/debugger/live-edit-breakpoints-expected.txt (143173 => 143174)


--- trunk/LayoutTests/inspector/debugger/live-edit-breakpoints-expected.txt	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/LayoutTests/inspector/debugger/live-edit-breakpoints-expected.txt	2013-02-18 09:30:22 UTC (rev 143174)
@@ -9,7 +9,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Editing source:
     TextEditor.removeBreakpoint(lineNumber = 2)
     TextEditor.addBreakpoint(lineNumber = 2, disabled = true)
@@ -17,7 +17,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Undoing source editing:
     OriginalTextEditor.removeBreakpoint(lineNumber = 2)
     TextEditor.removeBreakpoint(lineNumber = 2)
@@ -26,7 +26,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Finally removing breakpoint:
     TextEditor.removeBreakpoint(lineNumber = 2)
     Dumping breakpoint storage
@@ -38,7 +38,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Editing source:
     OriginalTextEditor.addBreakpoint(lineNumber = 2, disabled = false)
     TextEditor.removeBreakpoint(lineNumber = 2)
@@ -46,7 +46,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Committing edited source:
     OriginalTextEditor.removeBreakpoint(lineNumber = 2)
     TextEditor.removeBreakpoint(lineNumber = 2)
@@ -55,7 +55,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Finally removing breakpoint:
     TextEditor.removeBreakpoint(lineNumber = 2)
     Dumping breakpoint storage
@@ -67,7 +67,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Editing source:
     OriginalTextEditor.addBreakpoint(lineNumber = 2, disabled = false)
     TextEditor.removeBreakpoint(lineNumber = 2)
@@ -75,17 +75,17 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Committing edited source:
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Editing source again so that live edit could succeed:
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Committing edited source again:
     OriginalTextEditor.removeBreakpoint(lineNumber = 2)
     TextEditor.removeBreakpoint(lineNumber = 2)
@@ -94,7 +94,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Finally removing breakpoint:
     TextEditor.removeBreakpoint(lineNumber = 2)
     Dumping breakpoint storage
@@ -106,7 +106,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Editing source:
     OriginalTextEditor.addBreakpoint(lineNumber = 2, disabled = false)
     TextEditor.removeBreakpoint(lineNumber = 2)
@@ -114,17 +114,17 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Committing edited source:
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Undoing source editing:
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: debugger, project id: debugger:file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: debugger
 Committing edited source again:
     OriginalTextEditor.removeBreakpoint(lineNumber = 2)
     TextEditor.removeBreakpoint(lineNumber = 2)
@@ -133,7 +133,7 @@
     Dumping breakpoint storage
         edit-me-breakpoints.js:2, enabled:true
     Dumping breakpoint locations
-        url: edit-me-breakpoints.js, lineNumber: 2, project type: network, project id: file://
+        url: edit-me-breakpoints.js, lineNumber: 2, project name: network
 Finally removing breakpoint:
     TextEditor.removeBreakpoint(lineNumber = 2)
     Dumping breakpoint storage
@@ -148,15 +148,15 @@
     TextEditor.addBreakpoint(lineNumber = 2, disabled = false)
     Dumping breakpoint storage
     Dumping breakpoint locations
-        url: , lineNumber: 2, project type: debugger, project id: debugger:
+        url: , lineNumber: 2, project name: debugger
 Editing source:
     Dumping breakpoint storage
     Dumping breakpoint locations
-        url: , lineNumber: 2, project type: debugger, project id: debugger:
+        url: , lineNumber: 2, project name: debugger
 Undoing source editing:
     Dumping breakpoint storage
     Dumping breakpoint locations
-        url: , lineNumber: 2, project type: debugger, project id: debugger:
+        url: , lineNumber: 2, project name: debugger
 Finally removing breakpoint:
     TextEditor.removeBreakpoint(lineNumber = 2)
     Dumping breakpoint storage
@@ -171,15 +171,15 @@
     TextEditor.addBreakpoint(lineNumber = 2, disabled = false)
     Dumping breakpoint storage
     Dumping breakpoint locations
-        url: , lineNumber: 2, project type: debugger, project id: debugger:
+        url: , lineNumber: 2, project name: debugger
 Editing source:
     Dumping breakpoint storage
     Dumping breakpoint locations
-        url: , lineNumber: 2, project type: debugger, project id: debugger:
+        url: , lineNumber: 2, project name: debugger
 Undoing source editing:
     Dumping breakpoint storage
     Dumping breakpoint locations
-        url: , lineNumber: 2, project type: debugger, project id: debugger:
+        url: , lineNumber: 2, project name: debugger
 Finally removing breakpoint:
     TextEditor.removeBreakpoint(lineNumber = 2)
     Dumping breakpoint storage

Modified: trunk/LayoutTests/inspector/debugger/live-edit-breakpoints.html (143173 => 143174)


--- trunk/LayoutTests/inspector/debugger/live-edit-breakpoints.html	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/LayoutTests/inspector/debugger/live-edit-breakpoints.html	2013-02-18 09:30:22 UTC (rev 143174)
@@ -39,8 +39,8 @@
         for (var i = 0; i < locations.length; ++i) {
             var uiLocation = locations[i].uiLocation;
             var uiSourceCode = uiLocation.uiSourceCode;
-            var project = uiSourceCode.project();
-            InspectorTest.addResult("        url: " + pathToFileName(uiSourceCode.originURL()) + ", lineNumber: " + uiLocation.lineNumber + ", project type: " + project.type() + ", project id: " + project.id());
+            var project = WebInspector.workspace.projectForUISourceCode(uiSourceCode);
+            InspectorTest.addResult("        url: " + pathToFileName(uiSourceCode.originURL()) + ", lineNumber: " + uiLocation.lineNumber + ", project name: " + project.id());
         }
     }
 

Modified: trunk/LayoutTests/inspector/uisourcecode-revisions.html (143173 => 143174)


--- trunk/LayoutTests/inspector/uisourcecode-revisions.html	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/LayoutTests/inspector/uisourcecode-revisions.html	2013-02-18 09:30:22 UTC (rev 143174)
@@ -9,7 +9,7 @@
     function createMockProject()
     {
         var workspace = new WebInspector.Workspace();
-        var projectDelegate = new WebInspector.SimpleProjectDelegate("", WebInspector.projectTypes.Network);
+        var projectDelegate = new WebInspector.SimpleProjectDelegate(WebInspector.projectTypes.Network);
         var project = workspace.addProject(projectDelegate);
         project.requestFileContent = function(uri, callback)
         {

Modified: trunk/Source/WebCore/ChangeLog (143173 => 143174)


--- trunk/Source/WebCore/ChangeLog	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/Source/WebCore/ChangeLog	2013-02-18 09:30:22 UTC (rev 143174)
@@ -1,3 +1,32 @@
+2013-02-18  Sheriff Bot  <webkit.review....@gmail.com>
+
+        Unreviewed, rolling out r143100.
+        http://trac.webkit.org/changeset/143100
+        https://bugs.webkit.org/show_bug.cgi?id=110088
+
+        Breaks file system support in workspace. (Requested by vsevik
+        on #webkit).
+
+        * inspector/front-end/ScriptSnippetModel.js:
+        (WebInspector.ScriptSnippetModel.prototype._addScriptSnippet):
+        * inspector/front-end/SimpleWorkspaceProvider.js:
+        (WebInspector.SimpleProjectDelegate):
+        (WebInspector.SimpleProjectDelegate.prototype.id):
+        (WebInspector.SimpleProjectDelegate.prototype.displayName):
+        (WebInspector.SimpleProjectDelegate.prototype.addFile):
+        (WebInspector.SimpleWorkspaceProvider):
+        (WebInspector.SimpleWorkspaceProvider.uriForURL):
+        (WebInspector.SimpleWorkspaceProvider.prototype._innerAddFileForURL):
+        (WebInspector.SimpleWorkspaceProvider.prototype.removeFile):
+        (WebInspector.SimpleWorkspaceProvider.prototype.reset):
+        * inspector/front-end/UISourceCode.js:
+        (WebInspector.UISourceCode.prototype.uri):
+        * inspector/front-end/Workspace.js:
+        (WebInspector.Project.prototype._fileAdded):
+        (WebInspector.Project.prototype._fileRemoved):
+        (WebInspector.Project.prototype.uiSourceCodeForURI):
+        (WebInspector.Workspace.prototype.projectForUISourceCode):
+
 2013-02-15  Eugene Klyuchnikov  <eus...@chromium.org>
 
         Web Inspector: Introduce ProfilesPanelDescriptor.

Modified: trunk/Source/WebCore/inspector/front-end/ScriptSnippetModel.js (143173 => 143174)


--- trunk/Source/WebCore/inspector/front-end/ScriptSnippetModel.js	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/Source/WebCore/inspector/front-end/ScriptSnippetModel.js	2013-02-18 09:30:22 UTC (rev 143174)
@@ -82,7 +82,7 @@
      */
     _addScriptSnippet: function(snippet)
     {
-        var uiSourceCode = this._workspaceProvider.addFileByName("", snippet.name, new WebInspector.SnippetContentProvider(snippet), true);
+        var uiSourceCode = this._workspaceProvider.addFileForURL(snippet.name, new WebInspector.SnippetContentProvider(snippet), true);
         var scriptFile = new WebInspector.SnippetScriptFile(this, uiSourceCode);
         uiSourceCode.setScriptFile(scriptFile);
         this._snippetIdForUISourceCode.put(uiSourceCode, snippet.id);
@@ -103,7 +103,7 @@
         this._releaseSnippetScript(uiSourceCode);
         delete this._uiSourceCodeForSnippetId[snippet.id];
         this._snippetIdForUISourceCode.remove(uiSourceCode);
-        this._workspaceProvider.removeFileByName("", snippet.name);
+        this._workspaceProvider.removeFile(snippet.name);
     },
 
     /**

Modified: trunk/Source/WebCore/inspector/front-end/SimpleWorkspaceProvider.js (143173 => 143174)


--- trunk/Source/WebCore/inspector/front-end/SimpleWorkspaceProvider.js	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/Source/WebCore/inspector/front-end/SimpleWorkspaceProvider.js	2013-02-18 09:30:22 UTC (rev 143174)
@@ -33,28 +33,21 @@
  * @implements {WebInspector.ProjectDelegate}
  * @extends {WebInspector.Object}
  */
-WebInspector.SimpleProjectDelegate = function(name, type)
+WebInspector.SimpleProjectDelegate = function(type)
 {
-    this._name = name;
     this._type = type;
     /** @type {Object.<string, WebInspector.ContentProvider>} */
     this._contentProviders = {};
     this._lastUniqueSuffix = 0;
 }
 
-WebInspector.SimpleProjectDelegate.projectId = function(name, type)
-{
-    var typePrefix = type !== WebInspector.projectTypes.Network ? (type + ":") : "";
-    return typePrefix + name;
-}
-
 WebInspector.SimpleProjectDelegate.prototype = {
     /**
      * @return {string}
      */
     id: function()
     {
-        return WebInspector.SimpleProjectDelegate.projectId(this._name, this._type);
+        return this._type;
     },
 
     /**
@@ -70,18 +63,7 @@
      */
     displayName: function()
     {
-        if (typeof this._displayName !== "undefined")
-            return this._displayName;
-        if (!this._name) {
-            this._displayName = "";
-            return this._displayName;
-        }
-        var parsedURL = new WebInspector.ParsedURL(this._name);
-        if (parsedURL.isValid)
-            this._displayName = parsedURL.host + (parsedURL.port ? (":" + parsedURL.port) : "");
-        else
-            this._displayName = this._name;
-        return this._displayName;
+        return "";
     },
 
     /**
@@ -105,7 +87,6 @@
     },
 
     /**
-     * @param {string} path
      * @param {string} query
      * @param {boolean} caseSensitive
      * @param {boolean} isRegex
@@ -123,12 +104,12 @@
      * @param {WebInspector.ContentProvider} contentProvider
      * @param {boolean} isEditable
      * @param {boolean=} isContentScript
-     * @return {string}
      */
     addFile: function(path, forceUniquePath, url, contentProvider, isEditable, isContentScript)
     {
         if (forceUniquePath)
             path = this._uniquePath(path); 
+        console.assert(!this._contentProviders[path]);
         var fileDescriptor = new WebInspector.FileDescriptor(path, url, url, contentProvider.contentType(), isEditable, isContentScript);
         this._contentProviders[path] = contentProvider;
         this.dispatchEventToListeners(WebInspector.ProjectDelegate.Events.FileAdded, fileDescriptor);
@@ -175,84 +156,27 @@
 {
     this._workspace = workspace;
     this._type = type;
-    this._simpleProjectDelegates = {};
+    this._simpleProjectDelegate = new WebInspector.SimpleProjectDelegate(this._type);
+    this._workspace.addProject(this._simpleProjectDelegate);
 }
 
 /**
  * @param {string} url
- * @return {Array.<string>}
- */
-WebInspector.SimpleWorkspaceProvider.splitURL = function(url)
-{
-    var parsedURL = new WebInspector.ParsedURL(url);
-    var origin;
-    var folderPath;
-    var name;
-    if (parsedURL.isValid) {
-        origin = parsedURL.scheme + "://" + parsedURL.host;
-        if (parsedURL.port)
-            origin += ":" + parsedURL.port;
-        folderPath = parsedURL.folderPathComponents;
-        name = parsedURL.lastPathComponent;
-        if (parsedURL.queryParams)
-            name += "?" + parsedURL.queryParams;
-    } else {
-        origin = "";
-        folderPath = "";
-        name = url;
-    }
-    var result = [origin];
-    var splittedPath = folderPath.split("/");
-    for (var i = 1; i < splittedPath.length; ++i)
-        result.push(splittedPath[i]);
-    result.push(name);
-    return result;
-}
-
-/**
- * @param {Array.<string>} splittedURL
  * @return {string}
  */
-WebInspector.SimpleWorkspaceProvider._pathForSplittedURL = function(splittedURL)
-{
-    splittedURL.shift();
-    return splittedURL.join("/");
+WebInspector.SimpleWorkspaceProvider.uriForURL = function(url, type)
+{   
+    var uriTypePrefix = type !== WebInspector.projectTypes.Network ? (type + ":") : "";
+    var uri = uriTypePrefix + url;
+    return uri;
 }
 
-/**
- * @param {string} url
- * @param {string} type
- * @return {string}
- */
-WebInspector.SimpleWorkspaceProvider.uriForURL = function(url, type)
-{
-    var splittedURL = WebInspector.SimpleWorkspaceProvider.splitURL(url);
-    var projectId = WebInspector.SimpleProjectDelegate.projectId(splittedURL[0], type);
-    var path = WebInspector.SimpleWorkspaceProvider._pathForSplittedURL(splittedURL);
-    return WebInspector.UISourceCode.uri(projectId, path);
- }
-
 WebInspector.SimpleWorkspaceProvider.prototype = {
     /**
-     * @param {string} projectName
-     * @return {WebInspector.ProjectDelegate}
-     */
-    _projectDelegate: function(projectName)
-    {
-        if (this._simpleProjectDelegates[projectName])
-            return this._simpleProjectDelegates[projectName];
-        var simpleProjectDelegate = new WebInspector.SimpleProjectDelegate(projectName, this._type);
-        this._simpleProjectDelegates[projectName] = simpleProjectDelegate;
-        this._workspace.addProject(simpleProjectDelegate);
-        return simpleProjectDelegate;
-    },
- 
-    /**
      * @param {string} url
      * @param {WebInspector.ContentProvider} contentProvider
      * @param {boolean} isEditable
      * @param {boolean=} isContentScript
-     * @return {WebInspector.UISourceCode}
      */
     addFileForURL: function(url, contentProvider, isEditable, isContentScript)
     {
@@ -264,7 +188,6 @@
      * @param {WebInspector.ContentProvider} contentProvider
      * @param {boolean} isEditable
      * @param {boolean=} isContentScript
-     * @return {WebInspector.UISourceCode}
      */
     addUniqueFileForURL: function(url, contentProvider, isEditable, isContentScript)
     {
@@ -275,62 +198,27 @@
      * @param {string} url
      * @param {WebInspector.ContentProvider} contentProvider
      * @param {boolean} isEditable
-     * @param {boolean} forceUnique
      * @param {boolean=} isContentScript
-     * @return {WebInspector.UISourceCode}
      */
     _innerAddFileForURL: function(url, contentProvider, isEditable, forceUnique, isContentScript)
     {
-        var splittedURL = WebInspector.SimpleWorkspaceProvider.splitURL(url);
-        var projectName = splittedURL[0];
-        var path = WebInspector.SimpleWorkspaceProvider._pathForSplittedURL(splittedURL);
-        return this._innerAddFile(projectName, path, url, contentProvider, isEditable, forceUnique, isContentScript);
+        var path = WebInspector.SimpleWorkspaceProvider.uriForURL(url, this._type);
+        path = this._simpleProjectDelegate.addFile(path, forceUnique, url, contentProvider, isEditable, isContentScript);
+        return this._workspace.uiSourceCode(this._simpleProjectDelegate.id(), path);
     },
 
     /**
-     * @param {string} projectName
-     * @param {string} name
-     * @param {WebInspector.ContentProvider} contentProvider
-     * @param {boolean} isEditable
-     * @param {boolean=} isContentScript
-     * @return {WebInspector.UISourceCode}
+     * @param {string} url
      */
-    addFileByName: function(projectName, name, contentProvider, isEditable, isContentScript)
+    removeFile: function(url)
     {
-        return this._innerAddFile("", name, name, contentProvider, isEditable, false, isContentScript);
+        var path = WebInspector.SimpleWorkspaceProvider.uriForURL(url, this._type);
+        this._simpleProjectDelegate.removeFile(path);
     },
 
-    /**
-     * @param {string} projectName
-     * @param {string} path
-     * @param {WebInspector.ContentProvider} contentProvider
-     * @param {boolean} isEditable
-     * @param {boolean} forceUnique
-     * @param {boolean=} isContentScript
-     * @return {WebInspector.UISourceCode}
-     */
-    _innerAddFile: function(projectName, path, url, contentProvider, isEditable, forceUnique, isContentScript)
-    {
-        var projectDelegate = this._projectDelegate(projectName);
-        path = projectDelegate.addFile(path, forceUnique, url, contentProvider, isEditable, isContentScript);
-        return this._workspace.uiSourceCode(projectDelegate.id(), path);
-    },
-
-    /**
-     * @param {string} projectName
-     * @param {string} path
-     */
-    removeFileByName: function(projectName, path)
-    {
-        var projectDelegate = this._projectDelegate(projectName);
-        projectDelegate.removeFile(path);
-    },
-
     reset: function()
     {
-        for (var projectName in this._simpleProjectDelegates)
-            this._simpleProjectDelegates[projectName].reset();
-        this._simpleProjectDelegates = {};
+        this._simpleProjectDelegate.reset();
     },
     
     __proto__: WebInspector.Object.prototype

Modified: trunk/Source/WebCore/inspector/front-end/UISourceCode.js (143173 => 143174)


--- trunk/Source/WebCore/inspector/front-end/UISourceCode.js	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/Source/WebCore/inspector/front-end/UISourceCode.js	2013-02-18 09:30:22 UTC (rev 143174)
@@ -81,37 +81,6 @@
     SourceMappingChanged: "SourceMappingChanged",
 }
 
-/**
- * @param {string} projectId
- * @param {string} path
- * @return {string}
- */
-WebInspector.UISourceCode.uri = function(projectId, path)
-{
-    if (!projectId)
-        return path;
-    if (!path)
-        return projectId;
-    return projectId + "/" + path;
-}
-
-/**
- * @param {string} projectId
- * @param {string} uri
- * @return {?string}
- */
-WebInspector.UISourceCode.path = function(projectId, uri)
-{
-    if (!projectId)
-        return uri;
-    if (!uri.startsWith(projectId))
-        return null;
-    var path = uri.substr(projectId.length);
-    if (path.length && path[0] === "/")
-        path = path.substr(1);
-    return path;
-}
-
 WebInspector.UISourceCode.prototype = {
     /**
      * @return {string}
@@ -134,7 +103,7 @@
      */
     uri: function()
     {
-        return WebInspector.UISourceCode.uri(this._project.id(), this._path);
+        return this._path;
     },
 
     /**

Modified: trunk/Source/WebCore/inspector/front-end/Workspace.js (143173 => 143174)


--- trunk/Source/WebCore/inspector/front-end/Workspace.js	2013-02-18 09:27:42 UTC (rev 143173)
+++ trunk/Source/WebCore/inspector/front-end/Workspace.js	2013-02-18 09:30:22 UTC (rev 143174)
@@ -187,7 +187,7 @@
     _fileAdded: function(event)
     {
         var fileDescriptor = /** @type {WebInspector.FileDescriptor} */ (event.data);
-        var uiSourceCode = this.uiSourceCode(fileDescriptor.path);
+        var uiSourceCode = this.uiSourceCodeForURI(fileDescriptor.path);
         if (uiSourceCode) {
             // FIXME: Implement
             return;
@@ -201,7 +201,7 @@
     _fileRemoved: function(event)
     {
         var path = /** @type {string} */ (event.data);
-        var uiSourceCode = this.uiSourceCode(path);
+        var uiSourceCode = this.uiSourceCodeForURI(path);
         if (!uiSourceCode)
             return;
         delete this._uiSourceCodes[uiSourceCode.path()];
@@ -243,10 +243,7 @@
      */
     uiSourceCodeForURI: function(uri)
     {
-        var path = WebInspector.UISourceCode.path(this.id(), uri);
-        if (typeof path !== "string")
-            return null;
-        return this._uiSourceCodes[path];
+        return this.uiSourceCode(uri);
     },
 
     /**
@@ -298,6 +295,7 @@
 WebInspector.projectTypes = {
     Debugger: "debugger",
     LiveEdit: "liveedit",
+    Compiler: "compiler",
     Network: "network",
     Snippets: "snippets",
     FileSystem: "filesystem"
@@ -443,6 +441,19 @@
         return result;
     },
 
+    /**
+     * @return {?WebInspector.Project}
+     */
+    projectForUISourceCode: function(uiSourceCode)
+    {
+        for (var projectId in this._projects) {
+            var project = this._projects[projectId];
+            if (project.uiSourceCodeForURI(uiSourceCode.uri()))
+                return project;
+        }
+        return null;
+    },
+
     __proto__: WebInspector.Object.prototype
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to