DevTools: make StylesSourceMapping in charge of creating and removing 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 75 matching lines @@
               SDK.ResourceTreeModel.Events.MainFrameNavigated, this._mainFrameNavigated, this));
     }
 
     var debuggerModel = SDK.DebuggerModel.fromTarget(target);
     if (debuggerModel) {
       this._eventListeners.push(
           debuggerModel.addEventListener(SDK.DebuggerModel.Events.ParsedScriptSource, this._parsedScriptSource, this),
           debuggerModel.addEventListener(
               SDK.DebuggerModel.Events.FailedToParseScriptSource, this._parsedScriptSource, this));
     }
-    var cssModel = SDK.CSSModel.fromTarget(target);
-    if (cssModel) {
-      this._eventListeners.push(
-          cssModel.addEventListener(SDK.CSSModel.Events.StyleSheetAdded, this._styleSheetAdded, this),
-          cssModel.addEventListener(SDK.CSSModel.Events.StyleSheetRemoved, this._styleSheetRemoved, this));
-    }
     this._eventListeners.push(target.targetManager().addEventListener(
         SDK.TargetManager.Events.SuspendStateChanged, this._suspendStateChanged, this));
   }
 
   /**
    * @param {!SDK.Target} target
    * @param {?SDK.ResourceTreeFrame} frame
    * @param {boolean} isContentScripts
    * @return {string}
    */
@@ skipping 77 matching lines @@
    * @return {!Workspace.UISourceCode}
    */
   addFile(contentProvider, frame, isContentScript, contentSize) {
     var uiSourceCode = this._createFile(contentProvider, frame, isContentScript || false);
     var metadata = typeof contentSize === 'number' ? new Workspace.UISourceCodeMetadata(null, contentSize) : null;
     this._addUISourceCodeWithProvider(uiSourceCode, contentProvider, metadata);
     return uiSourceCode;
   }
 
   /**
+   * @param {string} url
    * @param {?SDK.ResourceTreeFrame} frame
-   * @param {string} url
+   * @param {boolean} isContentScript
+   * @return {?Workspace.UISourceCode}
    */
-  _removeFileForURL(frame, url) {
-    var project = this._workspaceProjects.get(Bindings.NetworkProject.projectId(this.target(), frame, false));
-    if (!project)
-      return;
-    project.removeFile(url);
+  getFile(url, frame, isContentScript) {

[dgozman, 2017/01/27 23:49:12]

uiSourceCode(frame, url, isContentScript)

+    var project = this._workspaceProject(frame, isContentScript);
+    return project.uiSourceCodeForURL(url);
+  }
+
+  /**
+   * @param {!Workspace.UISourceCode} uiSourceCode
+   */
+  removeFile(uiSourceCode) {

[dgozman, 2017/01/27 23:49:12]

removeUISourceCode

[lushnikov, 2017/01/30 14:19:15]

I was a bit hesitant to add just another one "removeUISourceCode" method

+    var project = uiSourceCode.project();
+    console.assert(
+        project[Bindings.NetworkProject._targetSymbol] === this.target(), 'Cannot remove foreign UISourceCode');
+    /** @type {!Bindings.ContentProviderBasedProject} */ (project).removeFile(uiSourceCode.url());
   }
 
   _populate() {
     /**
      * @param {!SDK.ResourceTreeFrame} frame
      * @this {Bindings.NetworkProject}
      */
     function populateFrame(frame) {
       for (var i = 0; i < frame.childFrames.length; ++i)
         populateFrame.call(this, frame.childFrames[i]);
@@ skipping 32 matching lines @@
       if (!parsedURL.isValid)
         return;
     }
     var uiSourceCode = this._createFile(script, SDK.ResourceTreeFrame.fromScript(script), script.isContentScript());
     uiSourceCode[Bindings.NetworkProject._scriptSymbol] = script;
     var resource = SDK.ResourceTreeModel.resourceForURL(uiSourceCode.url());
     this._addUISourceCodeWithProvider(uiSourceCode, script, this._resourceMetadata(resource));
   }
 
   /**
-   * @param {!Common.Event} event
+   * @param {!SDK.CSSStyleSheetHeader} header
+   * @return {!Workspace.UISourceCode}
    */
-  _styleSheetAdded(event) {
-    var header = /** @type {!SDK.CSSStyleSheetHeader} */ (event.data);
-    if (header.isInline && !header.hasSourceURL && header.origin !== 'inspector')
-      return;
-    if (!header.resourceURL())
-      return;
-
+  createFileForStyleSheetHeader(header) {

[dgozman, 2017/01/27 23:49:12]

Let's get rid of this one as well. We should expose something like
addUISourceCode(frame, contentProvider, isContentScript, mimeType).

     var originalContentProvider = header.originalContentProvider();
     var uiSourceCode = this._createFile(originalContentProvider, SDK.ResourceTreeFrame.fromStyleSheet(header), false);
     uiSourceCode[Bindings.NetworkProject._styleSheetSymbol] = header;
     var resource = SDK.ResourceTreeModel.resourceForURL(uiSourceCode.url());
     this._addUISourceCodeWithProvider(uiSourceCode, originalContentProvider, this._resourceMetadata(resource));
+    return uiSourceCode;
   }
 
   /**
-   * @param {!Common.Event} event
-   */
-  _styleSheetRemoved(event) {
-    var header = /** @type {!SDK.CSSStyleSheetHeader} */ (event.data);
-    if (header.isInline && !header.hasSourceURL && header.origin !== 'inspector')
-      return;
-
-    this._removeFileForURL(SDK.ResourceTreeFrame.fromStyleSheet(header), header.resourceURL());
-  }
-
-  /**
    * @param {!Common.Event} event
    */
   _resourceAdded(event) {
     var resource = /** @type {!SDK.Resource} */ (event.data);
     this._addResource(resource);
   }
 
   /**
    * @param {!SDK.Resource} resource
    */
@@ skipping 111 matching lines @@
     return workspace.uiSourceCode(Bindings.NetworkProject.projectId(header.target(), frame, false), url);
   }
 };
 
 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._frameSymbol = Symbol('frame');