DevTools: make CompilerScriptMapping / SASSSourceMapping manage UISourceCodes

third_party/WebKit/Source/devtools/front_end/bindings/NetworkProject.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 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 /**
  * @implements {SDK.TargetManager.Observer}
  * @unrestricted
  */
-Bindings.NetworkProjectManager = class {
+Bindings.NetworkProjectManager = class extends Common.Object {
   /**
    * @param {!SDK.TargetManager} targetManager
    * @param {!Workspace.Workspace} workspace
    */
   constructor(targetManager, workspace) {
+    super();
     this._workspace = workspace;
     targetManager.observeTargets(this);
   }
 
   /**
    * @override
    * @param {!SDK.Target} target
    */
   targetAdded(target) {
     new Bindings.NetworkProject(target, this._workspace, target.model(SDK.ResourceTreeModel));
   }
 
   /**
    * @override
    * @param {!SDK.Target} target
    */
   targetRemoved(target) {
     Bindings.NetworkProject.forTarget(target)._dispose();
   }
 };
 
+Bindings.NetworkProjectManager.Events = {
+  FrameAttributionChanged: Symbol('FrameAttributionChanged')

[dgozman, 2017/05/11 23:55:16]

Make it two events.

[lushnikov, 2017/05/12 01:18:00]

Done.

+};
+
 /**
  * @unrestricted
  */
 Bindings.NetworkProject = class {
   /**
    * @param {!SDK.Target} target
    * @param {!Workspace.Workspace} workspace
    * @param {?SDK.ResourceTreeModel} resourceTreeModel
    */
   constructor(target, workspace, resourceTreeModel) {
@@ skipping 50 matching lines @@
   /**
    * @param {!SDK.Target} target
    * @return {!Bindings.NetworkProject}
    */
   static forTarget(target) {
     return target[Bindings.NetworkProject._networkProjectSymbol];
   }
 
   /**
    * @param {!Workspace.UISourceCode} uiSourceCode
-   * @return {?Set<string>}
+   * @param {string} frameId
    */
-  static frameAttribution(uiSourceCode) {
-    var frameId = uiSourceCode[Bindings.NetworkProject._frameAttributionSymbol];
-    return frameId ? new Set([frameId]) : null;
+  static setInitialFrameAttribution(uiSourceCode, frameId) {

[dgozman, 2017/05/11 23:55:15]

Where is the "copy from" parameter?

[lushnikov, 2017/05/12 01:18:00]

It's not needed until we migrate ResourceScriptMapping on this.

+    var attribution = new Map();
+    attribution.set(frameId, 1);
+    uiSourceCode[Bindings.NetworkProject._frameAttributionSymbol] = attribution;
   }
 
   /**
+   * @param {!Workspace.UISourceCode} uiSourceCode
+   * @param {string} frameId
+   */
+  static addFrameAttribution(uiSourceCode, frameId) {
+    var frameAttribution = uiSourceCode[Bindings.NetworkProject._frameAttributionSymbol];
+    var count = frameAttribution.get(frameId) || 0;
+    frameAttribution.set(frameId, count + 1);
+    if (count !== 0)
+      return;
+
+    var target = Bindings.NetworkProject.targetForUISourceCode(uiSourceCode);
+    var resourceTreeModel = target && target.model(SDK.ResourceTreeModel);
+    var frame = resourceTreeModel ? resourceTreeModel.frameForId(frameId) : null;
+    if (!frame)
+      return;
+
+    var data = {uiSourceCode: uiSourceCode, added: frame, removed: null};
+    Bindings.networkProjectManager.dispatchEventToListeners(
+        Bindings.NetworkProjectManager.Events.FrameAttributionChanged, data);
+  }
+
+  /**
+   * @param {!Workspace.UISourceCode} uiSourceCode
+   * @param {string} frameId
+   */
+  static removeFrameAttribution(uiSourceCode, frameId) {
+    var frameAttribution = uiSourceCode[Bindings.NetworkProject._frameAttributionSymbol];
+    var count = frameAttribution.get(frameId);
+    console.assert(
+        count, `Failed to remove frame attribution for url = ${uiSourceCode.url()} and frameId = ${frameId}`);

[dgozman, 2017/05/11 23:55:16]

This is slowing things down unnecessary.

[lushnikov, 2017/05/12 01:18:00]

Done.

+    frameAttribution.set(frameId, count - 1);
+    if (count !== 1)
+      return;
+    frameAttribution.delete(frameId);
+    var target = Bindings.NetworkProject.targetForUISourceCode(uiSourceCode);
+    var resourceTreeModel = target && target.model(SDK.ResourceTreeModel);
+    var frame = resourceTreeModel ? resourceTreeModel.frameForId(frameId) : null;
+    if (!frame)

[dgozman, 2017/05/11 23:55:16]

This is sketchy - you will not send a removed event sometimes and it will stick.

[lushnikov, 2017/05/12 01:18:00]

Done.

+      return;
+
+    var data = {uiSourceCode: uiSourceCode, added: null, removed: frame};
+    Bindings.networkProjectManager.dispatchEventToListeners(
+        Bindings.NetworkProjectManager.Events.FrameAttributionChanged, data);
+  }
+
+  /**
    * @param {!Workspace.UISourceCode} uiSourceCode
    * @return {?SDK.Target} target
    */
   static targetForUISourceCode(uiSourceCode) {
     return uiSourceCode.project()[Bindings.NetworkProject._targetSymbol] || null;
   }
 
   /**
+   * @param {!Workspace.Project} project
+   * @param {!SDK.Target} target
+   */
+  static setTargetForProject(project, target) {
+    project[Bindings.NetworkProject._targetSymbol] = target;
+  }
+
+  /**
    * @param {!Workspace.UISourceCode} uiSourceCode
    * @return {!Array<!SDK.ResourceTreeFrame>}
    */
   static framesForUISourceCode(uiSourceCode) {
     var target = Bindings.NetworkProject.targetForUISourceCode(uiSourceCode);
     var resourceTreeModel = target && target.model(SDK.ResourceTreeModel);
-    var frameIds = Bindings.NetworkProject.frameAttribution(uiSourceCode);
-    if (!resourceTreeModel || !frameIds)
+    var attribution = uiSourceCode[Bindings.NetworkProject._frameAttributionSymbol];
+    if (!resourceTreeModel || !attribution)
       return [];
-    var frames = Array.from(frameIds).map(frameId => resourceTreeModel.frameForId(frameId));
+    var frames = Array.from(attribution.keys()).map(frameId => resourceTreeModel.frameForId(frameId));
     return frames.filter(frame => !!frame);
   }
 
   /**
    * @param {!Workspace.UISourceCode} uiSourceCode
    * @return {string}
    */
   static uiSourceCodeMimeType(uiSourceCode) {
     if (uiSourceCode[Bindings.NetworkProject._scriptSymbol] || uiSourceCode[Bindings.NetworkProject._styleSheetSymbol])
       return uiSourceCode.contentType().canonicalMimeType();
@@ skipping 19 matching lines @@
       return project;
 
     project = new Bindings.ContentProviderBasedProject(
         this._workspace, projectId, projectType, '', false /* isServiceProject */);
     project[Bindings.NetworkProject._targetSymbol] = this._target;
     this._workspaceProjects.set(projectId, project);
     return project;
   }
 
   /**
-   * @param {!Common.ContentProvider} contentProvider
-   * @param {string} frameId
-   * @param {boolean} isContentScript
-   * @param {?number} contentSize
-   * @return {!Workspace.UISourceCode}
-   */
-  addSourceMapFile(contentProvider, frameId, isContentScript, contentSize) {
-    var uiSourceCode = this._createFile(contentProvider, frameId, isContentScript || false);
-    var metadata = typeof contentSize === 'number' ? new Workspace.UISourceCodeMetadata(null, contentSize) : null;
-    this._addUISourceCodeWithProvider(uiSourceCode, contentProvider, metadata);
-    return uiSourceCode;
-  }
-
-  /**
-   * @param {string} url
-   * @param {string} frameId
-   * @param {boolean} isContentScript
-   */
-  removeSourceMapFile(url, frameId, isContentScript) {
-    this._removeFileForURL(url, frameId, isContentScript);
-  }
-
-  /**
    * @param {string} frameId
    * @param {string} url
    * @param {boolean} isContentScript
    */
   _removeFileForURL(url, frameId, isContentScript) {
     var project =
         this._workspaceProjects.get(Bindings.NetworkProject.projectId(this._target, frameId, isContentScript));
     if (!project)
       return;
     project.removeFile(url);
@@ skipping 181 matching lines @@
   /**
    * @param {!Common.ContentProvider} contentProvider
    * @param {string} frameId
    * @param {boolean} isContentScript
    * @return {!Workspace.UISourceCode}
    */
   _createFile(contentProvider, frameId, isContentScript) {
     var url = contentProvider.contentURL();
     var project = this._workspaceProject(frameId, isContentScript);
     var uiSourceCode = project.createUISourceCode(url, contentProvider.contentType());
-    uiSourceCode[Bindings.NetworkProject._frameAttributionSymbol] = frameId;
+    if (frameId)
+      Bindings.NetworkProject.setInitialFrameAttribution(uiSourceCode, frameId);
     return uiSourceCode;
   }
 
   /**
    * @param {string} frameId
    * @param {string} url
    * @return {?Workspace.UISourceCodeMetadata}
    */
   _fetchMetadata(frameId, url) {
     if (!this._resourceTreeModel)
@@ skipping 51 matching lines @@
 };
 
 Bindings.NetworkProject._networkProjectSymbol = Symbol('networkProject');
 Bindings.NetworkProject._resourceSymbol = Symbol('resource');
 Bindings.NetworkProject._scriptSymbol = Symbol('script');
 Bindings.NetworkProject._styleSheetSymbol = Symbol('styleSheet');
 Bindings.NetworkProject._targetSymbol = Symbol('target');
 Bindings.NetworkProject._frameIdSymbol = Symbol('frameid');
 
 Bindings.NetworkProject._frameAttributionSymbol = Symbol('Bindings.NetworkProject._frameAttributionSymbol');