[DevTools] Make resource tree model optional

third_party/WebKit/Source/devtools/front_end/bindings/StylesSourceMapping.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 26 matching lines @@
  */
 WebInspector.StylesSourceMapping = function(cssModel, workspace, networkMapping)
 {
     this._cssModel = cssModel;
     this._workspace = workspace;
     this._workspace.addEventListener(WebInspector.Workspace.Events.ProjectRemoved, this._projectRemoved, this);
     this._workspace.addEventListener(WebInspector.Workspace.Events.UISourceCodeAdded, this._uiSourceCodeAddedToWorkspace, this);
     this._workspace.addEventListener(WebInspector.Workspace.Events.UISourceCodeRemoved, this._uiSourceCodeRemoved, this);
     this._networkMapping = networkMapping;
 
-    cssModel.target().resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._unbindAllUISourceCodes, this);
-
-    this._cssModel.addEventListener(WebInspector.CSSModel.Events.StyleSheetChanged, this._styleSheetChanged, this);
     /** @type {!Map<string, !Map<string, !Map<string, !WebInspector.CSSStyleSheetHeader>>>} */
     this._urlToHeadersByFrameId = new Map();
     /** @type {!Map.<!WebInspector.UISourceCode, !WebInspector.StyleFile>} */
     this._styleFiles = new Map();
+
+    this._cssModel.addEventListener(WebInspector.CSSModel.Events.StyleSheetChanged, this._styleSheetChanged, this);
+    WebInspector.targetManager.addEventListener(WebInspector.TargetManager.Events.MainFrameNavigated, this._unbindAllUISourceCodes, this);

[dgozman, 2016/08/19 20:23:37]

This class wants to listen to specific resourceTreeModel, let's make it
explicit.

[eostroukhov, 2016/08/20 01:22:30]

Done.

 }
 
 WebInspector.StylesSourceMapping.ChangeUpdateTimeoutMs = 200;
 
 WebInspector.StylesSourceMapping.prototype = {
     /**
      * @override
      * @param {!WebInspector.CSSLocation} rawLocation
      * @return {?WebInspector.UILocation}
      */
@@ skipping 109 matching lines @@
      */
     _unbindUISourceCode: function(uiSourceCode)
     {
         var styleFile = this._styleFiles.get(uiSourceCode);
         if (!styleFile)
             return;
         styleFile.dispose();
         this._styleFiles.delete(uiSourceCode);
     },
 
-    _unbindAllUISourceCodes: function()
+    /**
+     * @param {!WebInspector.Event} event
+     */
+    _unbindAllUISourceCodes: function(event)
     {
+        if (event.data.target() !== this.target())
+            return;
         for (var styleFile of this._styleFiles.values())
             styleFile.dispose();
         this._styleFiles.clear();
         this._urlToHeadersByFrameId = new Map();
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _uiSourceCodeAddedToWorkspace: function(event)
@@ skipping 201 matching lines @@
     },
 
     dispose: function()
     {
         if (this._terminated)
             return;
         this._terminated = true;
         WebInspector.EventTarget.removeEventListeners(this._eventListeners);
     }
 }