Devtools: [WIP] Implement enhanced devtools extension language APIs

Source/devtools/front_end/workspace/Workspace.js

 /*
  * Copyright (C) 2012 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 555 matching lines @@
 {
     this._fileSystemMapping = fileSystemMapping;
     /** @type {!Object.<string, !WebInspector.Project>} */
     this._projects = {};
     this._hasResourceContentTrackingExtensions = false;
 }
 
 WebInspector.Workspace.Events = {
     UISourceCodeAdded: "UISourceCodeAdded",
     UISourceCodeRemoved: "UISourceCodeRemoved",
+    UISourceCodeEdited: "UISourceCodeEdited",
     UISourceCodeContentCommitted: "UISourceCodeContentCommitted",
     ProjectAdded: "ProjectAdded",
     ProjectRemoved: "ProjectRemoved"
 }
 
 WebInspector.Workspace.prototype = {
     /**
      * @return {!Array.<!WebInspector.UISourceCode>}
      */
     unsavedSourceCodes: function()
@@ skipping 37 matching lines @@
         for (var i = 0; i < projects.length; ++i) {
             var project = projects[i];
             var uiSourceCode = project.uiSourceCodeForOriginURL(originURL);
             if (uiSourceCode)
                 return uiSourceCode;
         }
         return null;
     },
 
     /**
+     * @param {string} url
+     * @return {?WebInspector.UISourceCode}
+     */
+    uiSourceCodeForFilePath: function(url) {

[pfeldman, 2015/08/13 21:15:47]

You should not distinguish resources from the filesystem project from the
network-provided ones.

[wes, 2015/08/14 00:55:06]

I'll double check something but at some point I added this because OpenResource
wasn't working on the URLs of files from workspaces. If that's not the case when
I remove this I'm 100% sure it's 100% unnecessary. (I thought I had to make
other changes, too, but they're not in the diff... which means either I'm
crazy/forgetful or someone else integrated them into master while I was working
and I'm really fortunate that it got done already. Doesn't really matter.) I'll
remove it and double check.

[wes, 2015/08/14 01:13:32]

Right! I found the other bit I added (the answer to my self questioning above if
forgetfulness!) - ExtensionServer._onGetPageResources had a line added to also
pickup the files from filesystem (aka workspace) projects. Previously extensions
could get events and stuff about them from some channels (events), but couldn't
get a list of extant ones (they were excluded from getPageResources - which I
suppose is true, they're not _technically_ part of the page). After adding that
and immersing myself in using workspaces I realized that there were a small few
other places where the extension API was incapable of dealing with FileSystem
(aka workspace) project file urls, and the addition of this helper was the
result.
It's also why I was hellbent on unifying the canonical URLs used to represent
filesystem files.

+        var items = this.uiSourceCodesForProjectType(WebInspector.projectTypes.FileSystem);
+        for (var i = 0; i < items.length; ++i) {
+            var item = items[i];
+            if (item.uri() === url)
+                return item;
+        }
+        return null;
+    },
+
+    /**
      * @param {string} type
      * @return {!Array.<!WebInspector.UISourceCode>}
      */
     uiSourceCodesForProjectType: function(type)
     {
         var result = [];
         for (var projectName in this._projects) {
             var project = this._projects[projectName];
             if (project.type() === type)
                 result = result.concat(project.uiSourceCodes());
@@ skipping 85 matching lines @@
         return this._hasResourceContentTrackingExtensions;
     },
 
     __proto__: WebInspector.Object.prototype
 }
 
 /**
  * @type {!WebInspector.Workspace}
  */
 WebInspector.workspace;