1041266 - Event pane doesn't display any events when a media element with controls attribute is present

Status: RESOLVED FIXED

Description

[Trevor Rowbotham [:rowbot]]

Attachment: debugger-event-pane-fails.html

Steps to Reproduce:
1) Open attached test case.
2) Open debugger and select the event pane.

Actual Results:
No events of any type are displayed in the event pane.

Expected Results:
All events on the page should be displayed.

Removing the controls attribute from the video element causes events to be displayed as expected.  This also affects audio elements.

Comment 1

[Trevor Rowbotham [:rowbot]]

Browser Console show the following:

Handler function DebuggerClient.requester threw an exception: Error: getDefinitionSite is only valid for function grips.
Stack: ObjectClient.prototype.getDefinitionSite<.before@resource://gre/modules/devtools/dbg-client.jsm:2205:1
DebuggerClient.requester/<@resource://gre/modules/devtools/dbg-client.jsm:329:24
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
EventListeners.prototype._getDefinitionSite@chrome://browser/content/devtools/debugger-controller.js:1749:5
EventListeners.prototype._getListeners/<@chrome://browser/content/devtools/debugger-controller.js:1722:36
TaskImpl_run@resource://gre/modules/Task.jsm:314:40
TaskImpl@resource://gre/modules/Task.jsm:275:3
createAsyncFunction/asyncFunction@resource://gre/modules/Task.jsm:249:14
DebuggerClient.requester/</<@resource://gre/modules/devtools/dbg-client.jsm:344:9
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:96:9
Request.prototype.emit@resource://gre/modules/devtools/dbg-client.jsm:1087:29
DebuggerClient.prototype.onPacket/<@resource://gre/modules/devtools/dbg-client.jsm:922:9
resolve@resource://gre/modules/devtools/deprecated-sync-thenables.js:40:40
then@resource://gre/modules/devtools/deprecated-sync-thenables.js:20:43
then@resource://gre/modules/devtools/deprecated-sync-thenables.js:58:9
DebuggerClient.prototype.onPacket@resource://gre/modules/devtools/dbg-client.jsm:861:1
LocalDebuggerTransport.prototype.send/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/transport/transport.js:545:11
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
EventLoop.prototype.enter@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:432:5
ThreadActor.prototype._pushThreadPause@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:602:5
ThreadActor.prototype.onInterrupt@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/actors/script.js:1776:7
DSC_onPacket@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/main.js:1223:15
LocalDebuggerTransport.prototype.send/<@resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/transport/transport.js:545:11
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/DevToolsUtils.js:83:14
Line: 2205, column: 0 DevToolsUtils.js:59

Comment 2

[Panos Astithas (he/him) [:past] (please ni?)]

That error is fixed in the latest nightly, but there is another one that happens once we are past that. Patch forthcoming.

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Properly deal with event handlers that the debugger cannot unwrap

I've added the same check as in _getAllEventListeners, which I was reluctant to add previously without a use case that needed it.

Comment 4

[Panos Astithas (he/him) [:past] (please ni?)]

Try: https://tbpl.mozilla.org/?tree=Try&rev=49a4cf4d3a8d

Comment 5

[Victor Porof [:vporof][:vp]]

Comment on attachment 8459579 [details] [diff] [review]
Properly deal with event handlers that the debugger cannot unwrap

Review of attachment 8459579 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/debugger/test/doc_native-event-handler.html
@@ +9,5 @@
> +      function initialSetup(event) {
> +        debugger;
> +      }
> +
> +  		window.addEventListener("load", function() {}, false);

Is this crazy indentation intentional?

Comment 6

[Panos Astithas (he/him) [:past] (please ni?)]

https://hg.mozilla.org/integration/fx-team/rev/7ff31c7b215e

Comment 7

[Panos Astithas (he/him) [:past] (please ni?)]

(In reply to Victor Porof [:vporof][:vp] from comment #5)
> Is this crazy indentation intentional?

Duh, that's what I get for copying a file with hard tabs in it. Fixed.

Comment 8

[Wes Kocher (:KWierso)]

https://hg.mozilla.org/mozilla-central/rev/7ff31c7b215e

Comment 9

[Panos Astithas (he/him) [:past] (please ni?)]

Comment on attachment 8459579 [details] [diff] [review]
Properly deal with event handlers that the debugger cannot unwrap

Approval Request Comment
[Feature/regressing bug #]: this has been a problem for a very long time
[User impact if declined]: the debugger events pane will not work in some cases
[Describe test coverage new/current, TBPL]: browser chrome test and manual testing in m-c
[Risks and why]: almost no risk, tiny change to a part of the code that only developers will ever interact with
[String/UUID change made/needed]: none

Comment 10

[Trevor Rowbotham [:rowbot]]

The patch seems to have fixed the problem for me. Thanks for the quick fix.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/91e719c14690