DevTools: remove SDK.ResourceTreeFrame.fromXXX methods

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

Index: third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js
diff --git a/third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js b/third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js
index 222e97ae5e330859882011ec519add7a7f741d3b..d4bdc706119ba4c12f7a914aebbfc0ed47a89c55 100644
--- a/third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js
+++ b/third_party/WebKit/Source/devtools/front_end/bindings/SASSSourceMapping.js
@@ -64,8 +64,7 @@ Bindings.SASSSourceMapping = class {
       var contentProvider = sourceMap.sourceContentProvider(sassURL, Common.resourceTypes.SourceMapStyleSheet);
       var embeddedContent = sourceMap.embeddedContentByURL(sassURL);
       var embeddedContentLength = typeof embeddedContent === 'string' ? embeddedContent.length : null;
-      this._networkProject.addSourceMapFile(
-          contentProvider, SDK.ResourceTreeFrame.fromStyleSheet(header), false, embeddedContentLength);

[dgozman, 2017/03/17 19:00:21]

I think you can cleanup SDK.ResourceTreeFrame.fromXXX methods now.

[lushnikov, 2017/03/17 21:21:47]

YAY! Done.

+      this._networkProject.addSourceMapFile(contentProvider, header.frameId, false, embeddedContentLength);
     }
     Bindings.cssWorkspaceBinding.updateLocations(header);
     this._sourceMapAttachedForTest(sourceMap);
@@ -75,8 +74,11 @@ Bindings.SASSSourceMapping = class {
    * @param {!Common.Event} event
    */
   _sourceMapDetached(event) {
-    var header = /** @type {!SDK.CSSStyleSheetHeader} */ (event.data);
+    var header = /** @type {!SDK.CSSStyleSheetHeader} */ (event.data.header);
+    var sourceMap = /** @type {!SDK.SourceMap} */ (event.data.sourceMap);
     Bindings.cssWorkspaceBinding.updateLocations(header);
+    for (var sassURL of sourceMap.sourceURLs())
+      this._networkProject.removeSourceMapFile(sassURL, header.frameId, false);
   }
 
   /**
@@ -127,4 +129,4 @@ Bindings.SASSSourceMapping = class {
   dispose() {
     Common.EventTarget.removeEventListeners(this._eventListeners);
   }
-};
+};