[DevTools] Explicitly require ResourceTreeModel

third_party/WebKit/Source/devtools/front_end/elements/ComputedStyleModel.js

 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @extends {WebInspector.Object}
  * @constructor
  */
 WebInspector.ComputedStyleModel = function()
 {
@@ skipping 17 matching lines @@
 
     /**
      * @return {?WebInspector.CSSModel}
      */
     cssModel: function()
     {
         return this._cssModel && this._cssModel.isEnabled() ? this._cssModel : null;
     },
 
     /**
+     * @return {?WebInspector.ResourceTreeModel}
+     */
+    resourceTreeModel: function()
+    {
+        return this._resourceTreeModel || null;

[pfeldman, 2016/07/27 17:54:35]

This does not sound right, css should not need resource tree nor should it
expose it.

[eostroukhov-old, 2016/08/05 18:32:03]

This method was left here by mistake, removed now.

Q: below, should I check for RTM separately from the rest?

+    },
+
+    /**
      * @param {!WebInspector.Event} event
      */
     _onNodeChanged: function(event)
     {
         this._node = /** @type {?WebInspector.DOMNode} */(event.data);
         this._updateTarget(this._node ? this._node.target() : null);
         this._onComputedStyleChanged(null);
     },
 
     /**
      * @param {?WebInspector.Target} target
      */
     _updateTarget: function(target)
     {
         if (this._target === target)
             return;
         if (this._targetEvents) {
             WebInspector.EventTarget.removeEventListeners(this._targetEvents);
             this._targetEvents = null;
         }
         this._target = target;
 
         var domModel = null;
-        var resourceTreeModel = null;
         if (target) {
             this._cssModel = WebInspector.CSSModel.fromTarget(target);
             domModel = WebInspector.DOMModel.fromTarget(target);
-            resourceTreeModel = target.resourceTreeModel;
+            this._resourceTreeModel = target.resourceTreeModel;
         }
 
-        if (this._cssModel && domModel && resourceTreeModel) {
+        if (this._cssModel && domModel && this._resourceTreeModel) {
             this._targetEvents = [
                 this._cssModel.addEventListener(WebInspector.CSSModel.Events.StyleSheetAdded, this._onComputedStyleChanged, this),
                 this._cssModel.addEventListener(WebInspector.CSSModel.Events.StyleSheetRemoved, this._onComputedStyleChanged, this),
                 this._cssModel.addEventListener(WebInspector.CSSModel.Events.StyleSheetChanged, this._onComputedStyleChanged, this),
                 this._cssModel.addEventListener(WebInspector.CSSModel.Events.MediaQueryResultChanged, this._onComputedStyleChanged, this),
                 this._cssModel.addEventListener(WebInspector.CSSModel.Events.PseudoStateForced, this._onComputedStyleChanged, this),
                 this._cssModel.addEventListener(WebInspector.CSSModel.Events.ModelWasEnabled, this._onComputedStyleChanged, this),
                 domModel.addEventListener(WebInspector.DOMModel.Events.DOMMutated, this._onDOMModelChanged, this),
-                resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameResized, this._onFrameResized, this),
+                this._resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameResized, this._onFrameResized, this),
             ];
         }
     },
 
      /**
      * @param {?WebInspector.Event} event
      */
     _onComputedStyleChanged: function(event)
     {
         delete this._computedStylePromise;
@@ skipping 73 matching lines @@
 /**
  * @constructor
  * @param {!WebInspector.DOMNode} node
  * @param {!Map.<string, string>} computedStyle
  */
 WebInspector.ComputedStyleModel.ComputedStyle = function(node, computedStyle)
 {
     this.node = node;
     this.computedStyle = computedStyle;
 }