1241602 - Add telemetry for DOMMouseScroll and MozMousePixelScroll usage

Status: RESOLVED FIXED

(Core :: DOM: Events, defect, P5)

Description

[Olli Pettay [:smaug][[e-mail]]]

I was chatting with rbyers from blink and would be great to get rid of both mousewheel and DOMMouseScroll/MozPixelScroll events.
But before adding even warnings, better to have some telemetry about the usage.
Unfortunately the usage is probably very high atm.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

> Unfortunately the usage is probably very high atm.

Yeah, I think so. I see a lot of web sites which explain how to handle mouse wheel on web apps are still introducing legacy events rather than wheel events. Sigh...

Comment 2

[張俊芝(ZHANG Junzhi)]

How about the result of the telemetry?

Comment 3

[Karl Dubost💡 :karlcow]

Attachment: Bug 1241602 - Adds telemetry for DOMMouseScroll usage. r=smaug

Comment 4

[Karl Dubost💡 :karlcow]

Attachment: Bug 1241602 - Typo fix.

Depends on D106413

Comment 5

[Karl Dubost💡 :karlcow]

mousewheel stats for blink. ~4% today
https://www.chromestatus.com/metrics/feature/timeline/popularity/1125

Libraries like CodeMirror use a

  // Listen to wheel events in order to try and update the viewport on time.
  on(d.scroller, "mousewheel", e => onScrollWheel(cm, e))
  on(d.scroller, "DOMMouseScroll", e => onScrollWheel(cm, e))

https://github.com/codemirror/CodeMirror/blob/8bc57f76383e62e1a03c7d97c9eac74493fdbedc/src/edit/CodeMirror.js#L193-L195

Comment 6

[Karl Dubost💡 :karlcow]

Attachment: Bug 1241602 - Adds telemetry for MozMousePixelScroll usage. r=smaug

Depends on D106413

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4d83d8c8dbee
Adds telemetry for DOMMouseScroll usage. r=smaug
https://hg.mozilla.org/integration/autoland/rev/02845f9654d2
Adds telemetry for MozMousePixelScroll usage. r=smaug

Comment 8

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/4d83d8c8dbee
https://hg.mozilla.org/mozilla-central/rev/02845f9654d2