1456218 - WebRender Mac: with WebRender ON, on bikeradar.com a sharp vertical scroll is not registered until it completes

Status: RESOLVED WORKSFORME

(Core :: Graphics: WebRender, defect, P1)

Description

[Mark]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:61.0) Gecko/20100101 Firefox/61.0
Build ID: 20180423100754

Steps to reproduce:

https://www.bikeradar.com/road/news/article/road-bike-of-the-year-for-2018-winner-52168/

Nightly 2018-04-23 Mac

WebRender ON: do a sharp vertical scroll of this page using a very quick two finger swipe on the trackpad. The page doesn't move until the swipe finishes

WebRender OFF: the page always moves correctly with the swipe.

I have tried disabling add-ons, no change.

Sometimes it's fiddly to make it happen, but I can see it ;)

Comment 1

[Kartikaya Gupta (email:[e-mail])]

I can't reproduce this on my mac using the latest nightly. Does it still happen if you set gfx.webrender.async-scene-build to 0 (browser restart required)?

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Not clear from comment 0 if this is a regression or not, it might be, in which case a regression range would still be useful.

Comment 3

[Mark]

Actually I think the problem is with an add-on:- Multi-Touch Zoom. This gives Safari-like pinch to zoom, it's nice. But the developer admits it's a clever hack. When I disable Multi-Touch Zoom and restart Firefox, my WebRender+swipe problem disappears.  The problem also disappears if I set gfx.webrender.async-scene-build to 0 and restart


WebRender     Multi-Touch Zoom    gfx.webrender.async-scene-build
  On             Enabled                        default                      -> scroll problem
  Off            Enabled                        default                      -> works OK      
  On             Disabled                       default                      -> works OK
  On             Enabled                        0                            -> works OK

Nightly restarted between tests

Comment 4

[Kartikaya Gupta (email:[e-mail])]

That's good to know, thanks! IIRC that add-on registers listeners on the root of the document which slows down APZ scrolling. I guess that doesn't play well with the async scene build codepaths. It sounds a lot like bug 1455759, and probably has the same root cause.

Comment 5

[Mark]

Yes it does sound like bug 1455759. Hopefully a fix for that will fix my issue.

I might check and see if I get 400 ms delay, I will let you know if I find anything out.

[ It's a shame that after soooo many years we still have to rely on hacky add-ons to get a nice pinch zoom... but I guess that's not your department ;) ]

Comment 6

[Mark]

I have tried to understand what's going on using perf.html Marker Chart. But I'm stuck.

Should I see one or a series of DispatchSynthMouseMove events each time I fling?

On the bikeradar page in questions I see almost no DOMEvent mousemove or DispatchSynthMouseMove events when I fling, whether I have WebRender on or off, or Multi-Touch Zoom enabled or disabled. With WebRender off and Multi-Touch Zoom disabled I did 10 up flings and 10 down flings in succession with Gecko Profiler running, when I looked at the perf.html I only see one DOMEvent mousemove and one DispatchSynthMouseMove right at the start (perhaps as I clicked the focus away from the Gecko Profiler button?). No others. And yet the page scrolls fine in this configuration.

Other pages I look at show at least one DispatchSynthMouseMove per fling.  https://www.bikeradar.com/road/news/article/road-bike-of-the-year-for-2018-winner-52168/ shows zero.

So perhaps the problem is with the page not with WebRender? But it scrolls fine with WebRender off. And even with WR on, it scrolls "eventually" ... just delayed.

Comment 7

[Kartikaya Gupta (email:[e-mail])]

Are you still seeing this issue on the latest Nightly?

Comment 8

[Mark]

No. Seems fixed, thanks.

Comment 9

[Kartikaya Gupta (email:[e-mail])]

Thanks!