844631 - Smooth scroll duration cannot be set differently for touchpad / mouse wheel

Status: NEW

(Core :: Panning and Zooming, defect)

Description

[Avi Halachmi (:avih)]

Smooth scroll duration for different event origins was introduced on bug 206438 (set to 200-400ms for mouse wheel, and 150ms for everything else), and fixed for linux/OSX on bug 737758.

However, at the time I didn't have a touchpad which generates pixel events, and now I do, and I see that the touchpad events uses the mouse wheel duration pref value (general.smoothScroll.mouseWheel.durationMinMS and surationMaxMS), while we should probably use lower values for the much more rapid pixel events of the touchpad, and probably use a pref of its own (probably the pixel events pref).

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

Should this be marked as blocking bug 206438?

Comment 2

[Avi Halachmi (:avih)]

(In reply to Jared Wein [:jaws] from comment #1)
> Should this be marked as blocking bug 206438?

Probably not.

It appears that the reason for this bug is the same reason as for bug 844653 (touch pad scroll distance depends on zoom). It seems that my touchpad doesn't generate pixel events, but rather very rapid fractional-line events (windows was designed to accommodate such approach, see the discussion at bug 844653).

So Firefox receives mouse-wheel-lines scroll events, and according to that, uses the correct smooth scroll duration.

Initially I guessed that Firefox receives pixel scroll events, but it ends up using the mouse wheel duration pref, but that's not the case, so bug 206438 shouldn't be blocked.