[DevTools] Event Listeners Sidebar shows window listeners

Source/devtools/protocol.json

Index: Source/devtools/protocol.json
diff --git a/Source/devtools/protocol.json b/Source/devtools/protocol.json
index 78344e8f3b0ad5d504f6a258c59e2d1e0cd1ebf9..3e718a3851fd5ceb911de046d92ff2eaf285c3dc 100644
--- a/Source/devtools/protocol.json
+++ b/Source/devtools/protocol.json
@@ -2132,7 +2132,7 @@
                     { "name": "type", "type": "string", "description": "<code>EventListener</code>'s type." },
                     { "name": "useCapture", "type": "boolean", "description": "<code>EventListener</code>'s useCapture." },
                     { "name": "isAttribute", "type": "boolean", "description": "<code>EventListener</code>'s isAttribute." },
-                    { "name": "nodeId", "$ref": "NodeId", "description": "Target <code>DOMNode</code> id." },

[pfeldman, 2015/03/30 14:36:20]

Why would it be in DOM domain? For example, it might make sense for worjers
where there is no DOM. Wdyt?

[kozy, 2015/03/31 20:09:15]

I suppose that supporting it for workers should be separated issue and I don't
want to divide this method into two parts because it seems like there is no real
case when we want to receive only window or only node event listeners.
I can add enum for EventListener that describes its type: Node or window if in
our codebase we use empty nodeId for document.
Wdyt?

+                    { "name": "nodeId", "$ref": "NodeId", "optional": true, "description": "If present contains target <code>DOMNode</code> id otherwise target is window." },
                     { "name": "location", "$ref": "Debugger.Location", "description": "Handler code location." },
                     { "name": "handler", "$ref": "Runtime.RemoteObject", "optional": true, "description": "Event handler function value." }
                 ],