[DevTools] Support JQuery event listeners

Source/devtools/front_end/components/EventListenersView.js

 // Copyright 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.
 
 /**
+* @typedef {Array<{object: !WebInspector.RemoteObject, eventListeners: ?Array<!WebInspector.EventListener>, frameworkEventListeners: ?{eventListeners: ?Array<!WebInspector.EventListener>, internalHandlers: ?WebInspector.RemoteArray}, isInternal: ?Array<boolean>}>}
+*/
+WebInspector.EventListenersResult;
+
+/**
  * @constructor
  * @param {!Element} element
  */
 WebInspector.EventListenersView = function(element)
 {
     this._element = element;
     this._treeOutline = new TreeOutlineInShadow("event-listener-tree");
     this._treeOutline.registerRequiredCSS("components/objectValue.css");
     this._treeOutline.registerRequiredCSS("components/eventListenersView.css");
     this._treeOutline.setComparator(WebInspector.EventListenersTreeElement.comparator);
     this._treeOutline.element.classList.add("monospace");
     this._element.appendChild(this._treeOutline.element)
     this._emptyHolder = createElementWithClass("div", "info");
     this._emptyHolder.textContent = WebInspector.UIString("No Event Listeners");
     this._linkifier = new WebInspector.Linkifier();
     /** @type {!Map<string, !WebInspector.EventListenersTreeElement>} */
     this._treeItemMap = new Map();
 }
 
 WebInspector.EventListenersView.prototype = {
     /**
      * @param {!Array<!WebInspector.RemoteObject>} objects
      * @return {!Promise<undefined>}
      */
     addObjects: function(objects)
     {
         var promises = [];
-        for (var i = 0; i < objects.length; ++i)
-            promises.push(objects[i].eventListeners());
-        return Promise.all(promises).then(listenersCallback.bind(this));
-        /**
-         * @param {!Array<?Array<!WebInspector.EventListener>>} listeners
+        var results = [];
+        for (var i = 0; i < objects.length; ++i) {
+            var result = {object: objects[i]};

[pfeldman, 2015/09/03 18:56:35]

Why are you creating these one-time objects? Is this because you want to use
promises?

+            results.push(result);
+            // Fill result object with eventListeners and frameworkEventListeners.
+            promises.push(objects[i].eventListeners().then(storeResultTo.bind(null, result, "eventListeners")));

[pfeldman, 2015/09/03 18:56:35]

Please delete this utility function and do not use.

+            promises.push(WebInspector.EventListener.frameworkEventListeners(objects[i])
+                                      .then(storeResultTo.bind(null, result, "frameworkEventListeners")));
+        }
+        return Promise.all(promises).then(listenersCallback.bind(this, results));
+         /**
+         * @param {!WebInspector.EventListenersResult} results
          * @this {WebInspector.EventListenersView}
          */
-        function listenersCallback(listeners)
+        function listenersCallback(results)

[pfeldman, 2015/09/03 18:56:35]

Please do not append "Callback" to the end of every function. What does this
function do? Lets name it accordingly.

         {
             this.reset();
-            for (var i = 0; i < listeners.length; ++i)
-                this._addObjectEventListeners(objects[i], listeners[i]);
+
+            // set isInternal flag for framework internal event listeners
+            var promises = [];
+            for (var result of results) {
+                var frameworkInternalHandlers = result.frameworkEventListeners.internalHandlers;
+                if (frameworkInternalHandlers) {
+                    var listenersHandler = result.eventListeners.map(handlerArgument);
+                    promises.push(frameworkInternalHandlers.object().callFunctionJSONPromise(internalFlags, listenersHandler)

[pfeldman, 2015/09/03 18:56:35]

Why is there asynchrony even after you have event listeners array? I were saying
UI does not have asynchrony.

+                                                                    .then(setIsInternal.bind(null, result.eventListeners)));
+                }
+            }
+
+            /**
+             * @param {!WebInspector.EventListener} listener
+             * @return {!RuntimeAgent.CallArgument}
+             */
+            function handlerArgument(listener)
+            {
+                return WebInspector.RemoteObject.toCallArgument(listener.handler());

[pfeldman, 2015/09/03 18:56:35]

Why do we have to do this again?

+            }
+
+            /**
+             * @suppressReceiverCheck
+             * @return {!Array<boolean>}
+             * @this {Array<*>}
+             */
+            function internalFlags()
+            {
+                var isInternal = [];
+                var internalHandlersSet = new Set(this);
+                for (var handler of arguments)
+                    isInternal.push(internalHandlersSet.has(handler));
+                return isInternal;
+            }
+
+            /**
+             * @param {!Array<!WebInspector.EventListener>} eventListeners
+             * @param {!Array<boolean>} isInternal
+             */
+            function setIsInternal(eventListeners, isInternal)
+            {
+                for (var i = 0; i < eventListeners.length; ++i) {
+                    if (isInternal[i])
+                        eventListeners[i].setListenerType("frameworkInternal");
+                }
+            }
+
+            return Promise.all(promises).then(addResults.bind(this, results));
+        }
+
+        /**
+         * @param {!WebInspector.EventListenersResult} results
+         * @this {WebInspector.EventListenersView}
+         */
+        function addResults(results)
+        {
+            for (var result of results) {
+                this._addObjectEventListeners(result.object, result.eventListeners);
+                this._addObjectEventListeners(result.object, result.frameworkEventListeners.eventListeners);
+            }
             this.addEmptyHolderIfNeeded();
             this._eventListenersArrivedForTest();
+         }
+     },
+
+    /**
+     * @param {boolean} showFramework
+     */
+    showFrameworkListeners: function(showFramework)
+    {
+        var eventTypes = this._treeOutline.rootElement().children();
+        for (var eventType of eventTypes) {
+            for (var listenerElement of eventType.children()) {
+                var listenerType = listenerElement.eventListener().listenerType();
+                var hidden = false;
+                if (listenerType === "frameworkUser" && !showFramework)
+                    hidden = true;
+                if (listenerType === "frameworkInternal" && showFramework)
+                    hidden = true;
+                listenerElement.hidden = hidden;
+            }
         }
     },
 
     /**
      * @param {!WebInspector.RemoteObject} object
      * @param {?Array<!WebInspector.EventListener>} eventListeners
      */
     _addObjectEventListeners: function(object, eventListeners)
     {
         if (!eventListeners)
@@ skipping 110 matching lines @@
      * @param {!WebInspector.Linkifier} 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.sourceURL()));
         title.appendChild(WebInspector.ObjectPropertiesSection.createValueElement(object, false));
     },
 
+    /**
+     * @return {!WebInspector.EventListener}
+     */
+    eventListener: function()
+    {
+        return this._eventListener;
+    },
+
     __proto__: TreeElement.prototype
-}
+}