Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1403)

Unified Diff: Source/devtools/front_end/components/EventListenersView.js

Issue 1154703005: [DevTools] Added window listeners to EventListenersSidebar (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@extract-event-listeners-tree-outline
Patch Set: Test fixed Created 5 years, 7 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
Index: Source/devtools/front_end/components/EventListenersView.js
diff --git a/Source/devtools/front_end/components/EventListenersView.js b/Source/devtools/front_end/components/EventListenersView.js
index 8f45e57b14da677da16cc5a818a418aefd1a65c7..684087173aec872a7873a1d9193c4f30b9bf7b76 100644
--- a/Source/devtools/front_end/components/EventListenersView.js
+++ b/Source/devtools/front_end/components/EventListenersView.js
@@ -5,12 +5,10 @@
/**
* @constructor
* @param {!Element} element
- * @param {string} objectGroup
*/
-WebInspector.EventListenersView = function(element, objectGroup)
+WebInspector.EventListenersView = function(element)
{
this._element = element;
- this._objectGroup = objectGroup;
this._treeOutline = new TreeOutlineInShadow("event-listener-tree");
this._treeOutline.registerRequiredCSS("components/objectValue.css");
this._treeOutline.registerRequiredCSS("components/eventListenersView.css");
@@ -27,16 +25,16 @@ WebInspector.EventListenersView = function(element, objectGroup)
WebInspector.EventListenersView.prototype = {
/**
- * @param {!WebInspector.DOMNode} node
+ * @param {!WebInspector.RemoteObject} object
* @param {?Array<!WebInspector.EventListener>} eventListeners
*/
- _addNodeEventListeners: function(node, eventListeners)
+ _addObjectEventListeners: function(object, eventListeners)
{
if (!eventListeners)
return;
for (var eventListener of eventListeners) {
var treeItem = this._getOrCreateTreeElementForType(eventListener.type());
- treeItem.addNodeEventListener(eventListener, node);
+ treeItem.addObjectEventListener(eventListener, object);
}
},
@@ -57,41 +55,12 @@ WebInspector.EventListenersView.prototype = {
},
/**
- * @param {!WebInspector.DOMNode} node
- * @return {!Promise}
+ * @param {!WebInspector.RemoteObject} object
+ * @return {!Promise<undefined>}
*/
- addNodeEventListeners: function(node)
+ addObjectEventListeners: function(object)
{
- return new Promise(addEventListeners.bind(this));
- /**
- * @param {function(?)} fulfill
- * @param {function(*)} reject
- * @this {WebInspector.EventListenersView}
- */
- function addEventListeners(fulfill, reject)
- {
- node.resolveToObject(this._objectGroup, objectCallback.bind(this));
- /**
- * @param {?WebInspector.RemoteObject} object
- * @this {WebInspector.EventListenersView}
- */
- function objectCallback(object)
- {
- if (object)
- object.getEventListeners(listenersCallback.bind(this));
- else
- reject(undefined);
- }
- /**
- * @param {?Array<!WebInspector.EventListener>} listeners
- * @this {WebInspector.EventListenersView}
- */
- function listenersCallback(listeners)
- {
- this._addNodeEventListeners(node, listeners);
- fulfill(undefined);
- }
- }
+ return object.getEventListeners().then(this._addObjectEventListeners.bind(this, object));
pfeldman 2015/05/25 16:25:16 eventListeners()
kozy 2015/05/25 18:03:30 Done.
},
reset: function()
@@ -131,11 +100,11 @@ WebInspector.EventListenersTreeElement.comparator = function(element1, element2)
WebInspector.EventListenersTreeElement.prototype = {
/**
* @param {!WebInspector.EventListener} eventListener
- * @param {!WebInspector.DOMNode} node
+ * @param {!WebInspector.RemoteObject} object
*/
- addNodeEventListener: function(eventListener, node)
+ addObjectEventListener: function(eventListener, object)
{
- var treeElement = new WebInspector.NodeEventListenerBar(eventListener, node, this._linkifier);
+ var treeElement = new WebInspector.ObjectEventListenerBar(eventListener, object, this._linkifier);
this.appendChild(/** @type {!TreeElement} */ (treeElement));
},
@@ -146,16 +115,18 @@ WebInspector.EventListenersTreeElement.prototype = {
* @constructor
* @extends {TreeElement}
* @param {!WebInspector.EventListener} eventListener
+ * @param {!WebInspector.RemoteObject} object
* @param {!WebInspector.Linkifier} linkifier
*/
-WebInspector.EventListenerBar = function(eventListener, linkifier)
+WebInspector.ObjectEventListenerBar = function(eventListener, object, linkifier)
{
TreeElement.call(this, "", true);
this._eventListener = eventListener;
this.editable = false;
+ this._setTitle(object, linkifier);
}
-WebInspector.EventListenerBar.prototype = {
+WebInspector.ObjectEventListenerBar.prototype = {
onpopulate: function()
{
var properties = [];
@@ -167,42 +138,17 @@ WebInspector.EventListenerBar.prototype = {
WebInspector.ObjectPropertyTreeElement.populateWithProperties(this, properties, [], true, null);
},
- __proto__: TreeElement.prototype
-}
-
-/**
- * @constructor
- * @extends {WebInspector.EventListenerBar}
- * @param {!WebInspector.EventListener} eventListener
- * @param {!WebInspector.DOMNode} node
- * @param {!WebInspector.Linkifier} linkifier
- */
-WebInspector.NodeEventListenerBar = function(eventListener, node, linkifier)
-{
- WebInspector.EventListenerBar.call(this, eventListener, linkifier);
- this._setNodeTitle(node, linkifier);
-}
-
-WebInspector.NodeEventListenerBar.prototype = {
/**
- * @param {!WebInspector.DOMNode} node
+ * @param {!WebInspector.RemoteObject} object
* @param {!WebInspector.Linkifier} linkifier
*/
- _setNodeTitle: function(node, linkifier)
+ _setTitle: function(object, linkifier)
{
var title = this.listItemElement.createChild("span");
var subtitle = this.listItemElement.createChild("span", "event-listener-tree-subtitle");
subtitle.appendChild(linkifier.linkifyRawLocation(this._eventListener.location(), this._eventListener.sourceName()));
- if (node.nodeType() === Node.DOCUMENT_NODE) {
- title.textContent = "document";
- return;
- }
- if (this._eventListener.isSelected) {
- title.textContent = WebInspector.DOMPresentationUtils.simpleSelector(node);
- return;
- }
- title.appendChild(WebInspector.DOMPresentationUtils.linkifyNodeReference(node));
+ title.appendChild(WebInspector.ObjectPropertiesSection.createValueElement(object, false));
},
- __proto__: WebInspector.EventListenerBar.prototype
-}
+ __proto__: TreeElement.prototype
+}

Powered by Google App Engine
This is Rietveld 408576698