1803538 - Add scrollend event in event breakpoints panel

Status: RESOLVED FIXED

(DevTools :: Debugger, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

The scrollend event will be enabled by default in Bug 1803435 , it would be nice to show it in the event breakpoints panel

Comment 1

[Dan Robertson (:dlrobertson)]

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #0)

The scrollend event will be enabled by default in Bug 1803435 , it would be nice to show it in the event breakpoints panel

Thanks for creating this! Are we able to add this to the panel before the preference is enabled by default? I've never interacted with the breakpoints panel before, so I'm just curious what happens in the case that apz.scrollend-event.content.enabled were to be false (scrollend are no longer fired to content, but still fired to the browser chrome).

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

(In reply to Dan Robertson (:dlrobertson) from comment #1)

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #0)

The scrollend event will be enabled by default in Bug 1803435 , it would be nice to show it in the event breakpoints panel

Thanks for creating this! Are we able to add this to the panel before the preference is enabled by default? I've never interacted with the breakpoints panel before, so I'm just curious what happens in the case that apz.scrollend-event.content.enabled were to be false (scrollend are no longer fired to content, but still fired to the browser chrome).

I guess we would only show the item on the list if the pref is enabled so it's not confusing for people.
Which makes me realize, we don't need to wait for the feature to be enabled and can start looking into it right now

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1803538 - [devtools] Add scrollend in debugger event breakpoint panel. r=bomsy.

The event is only supported when apz.scrollend-event.content.enabled is set to
true, so we only show it in the debugger when it's supported.

A test is added to ensure the event breakpoint does work as expected.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2e7a46447741
[devtools] Add scrollend in debugger event breakpoint panel. r=bomsy.

Comment 5

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/2e7a46447741