Open Bug 1740164 Opened 3 years ago Updated 3 years ago

Transmogrify ScrollToCSSPixels to ScrollByCSSPixels even if there's no animation in APZ

Categories

(Core :: Panning and Zooming, defect, P3)

defect

Tracking

()

People

(Reporter: hiro, Unassigned)

References

Details

In bug 1692708 we are going to change ScrollToCSSPixels to a relative operation if there's any on-going animation in APZ triggered by user. In this bug we are going to apply it in the case of no animation as well.

Severity: -- → S3
Priority: -- → P3

Do you know if this additional change is needed for the Slack case, or is bug 1692708 is sufficient for that?

I think bug 1692708 is sufficient in most cases including Slack basically. This is literally for a race condition where APZ just starts an async scrolling but the corresponding RepaintRequest hasn't yet reached to the content's main-thread, at that moment if JS on the main-thread calls scrollTop, our relative scroll machinery won't be triggered. This bug is for the race case.

So, I am thinking that we can defer this until someone reports to us there's a wobbly scrolling site in the wild.

You need to log in before you can comment on or make changes to this bug.