DevTools: remove SDK.ResourceTreeFrame.fromXXX methods

third_party/WebKit/Source/devtools/front_end/sdk/CSSModel.js

Index: third_party/WebKit/Source/devtools/front_end/sdk/CSSModel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sdk/CSSModel.js b/third_party/WebKit/Source/devtools/front_end/sdk/CSSModel.js
index 088c0ebf21399985fb0c58f91bd6893887124666..50d2d98d2d26d52524f27a153f6faa13315333df 100644
--- a/third_party/WebKit/Source/devtools/front_end/sdk/CSSModel.js
+++ b/third_party/WebKit/Source/devtools/front_end/sdk/CSSModel.js
@@ -221,9 +221,10 @@ SDK.CSSModel = class extends SDK.SDKModel {
     if (!header.sourceMapURL || !this._sourceMapURLToHeaders.hasValue(header.sourceMapURL, header))
       return;
     this._sourceMapURLToHeaders.remove(header.sourceMapURL, header);
+    var sourceMap = this._sourceMapByURL.get(header.sourceMapURL);
     if (!this._sourceMapURLToHeaders.has(header.sourceMapURL))
       this._sourceMapByURL.delete(header.sourceMapURL);
-    this.dispatchEventToListeners(SDK.CSSModel.Events.SourceMapDetached, header);
+    this.dispatchEventToListeners(SDK.CSSModel.Events.SourceMapDetached, {header: header, sourceMap: sourceMap});

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

This removes source map sources whenever first header goes away. For example,
shadow dom stuff breaks.

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

I started writing tests and it turned out that we already are not doing proper
job in resolving sourcemap urls. I'll address the issue separately; let's land
the frameId usage for now

   }
 
   /**