Source/devtools/front_end/elements/MetricsSidebarPane.js - Issue 1104163003: Devtools: [ElementsPanel] Add dom listeners in sidebars

Unified Diff: Source/devtools/front_end/elements/MetricsSidebarPane.js

Issue 1104163003: Devtools: [ElementsPanel] Add dom listeners in sidebars (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@move-force-state

Patch Set: Created 5 years, 8 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « Source/devtools/front_end/elements/ElementsPanel.js ('k') | Source/devtools/front_end/elements/PlatformFontsSidebarPane.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: Source/devtools/front_end/elements/MetricsSidebarPane.js

diff --git a/Source/devtools/front_end/elements/MetricsSidebarPane.js b/Source/devtools/front_end/elements/MetricsSidebarPane.js

index 444c8c94e3066a75183568caba32e059e78ec119..fbc7b33ca4edcd22f5e9a277cd08bd714b379b66 100644

--- a/Source/devtools/front_end/elements/MetricsSidebarPane.js

+++ b/Source/devtools/front_end/elements/MetricsSidebarPane.js

@@ -60,6 +60,9 @@ WebInspector.MetricsSidebarPane.prototype = {

this._cssModel.removeEventListener(WebInspector.CSSStyleModel.Events.StyleSheetChanged, this.update, this);

this._cssModel.removeEventListener(WebInspector.CSSStyleModel.Events.MediaQueryResultChanged, this.update, this);

this._cssModel.removeEventListener(WebInspector.CSSStyleModel.Events.PseudoStateForced, this.update, this);

+ this._domModel.removeEventListener(WebInspector.DOMModel.Events.NodeInserted, this.update, this);

pfeldman 2015/04/28 05:53:20 Can we extract a helper object that would listen t

sergeyv 2015/04/28 08:53:25 Done.

+ this._domModel.removeEventListener(WebInspector.DOMModel.Events.NodeRemoved, this.update, this);

+ this._domModel.removeEventListener(WebInspector.DOMModel.Events.DistributedNodesChanged, this.update, this);

this._domModel.removeEventListener(WebInspector.DOMModel.Events.AttrModified, this._attributesUpdated, this);

this._domModel.removeEventListener(WebInspector.DOMModel.Events.AttrRemoved, this._attributesUpdated, this);

this._target.resourceTreeModel.removeEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameResized, this.update, this);

@@ -73,6 +76,9 @@ WebInspector.MetricsSidebarPane.prototype = {

this._cssModel.addEventListener(WebInspector.CSSStyleModel.Events.StyleSheetChanged, this.update, this);

this._cssModel.addEventListener(WebInspector.CSSStyleModel.Events.MediaQueryResultChanged, this.update, this);

this._cssModel.addEventListener(WebInspector.CSSStyleModel.Events.PseudoStateForced, this.update, this);

+ this._domModel.addEventListener(WebInspector.DOMModel.Events.NodeInserted, this.update, this);

+ this._domModel.addEventListener(WebInspector.DOMModel.Events.NodeRemoved, this.update, this);

+ this._domModel.addEventListener(WebInspector.DOMModel.Events.DistributedNodesChanged, this.update, this);

this._domModel.addEventListener(WebInspector.DOMModel.Events.AttrModified, this._attributesUpdated, this);

this._domModel.addEventListener(WebInspector.DOMModel.Events.AttrRemoved, this._attributesUpdated, this);

this._target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameResized, this.update, this);