on scroll event have a page visual update lag
Categories
(Core :: Layout, defect)
Tracking
()
People
(Reporter: karlcow, Unassigned)
References
(Blocks 1 open bug, Regression, )
Details
(Keywords: regression)
I'm not sure if it's graphics the issue or if it's because of intense getComputedPosition
- With Firefox Nightly on Android
- Go to https://m.hoyolab.com/genshin/#/home/0
- Then scroll
Expected:
Smooth scrool
Actual:
- Scroll janky
- Layout disappears and then reappears.
Performance issue?
Working fine on chrome.
Reporter | ||
Comment 1•4 years ago
|
||
A profile https://share.firefox.dev/38OFXPv
Check in between 18 and 23s
Comment 2•4 years ago
|
||
The renderer thread does appear to be doing anything for the first 2 seconds of that period, which would certainly explain the jank. It's possible that webrender is taking too long further back in the pipeline. Karl, could you take another profile using the "Firefox Graphics" preset?
Could you also try setting gfx.webrender.force-disabled to true, restart, and see if that helps?
Reporter | ||
Comment 3•4 years ago
|
||
Jamie,
Yeah sure.
Firefox Graphics on with two subsequent scrolls.
https://share.firefox.dev/3cRifTE
And indeed setting gfx.webrender.force-disabled
to true
solves the issue
Updated•4 years ago
|
Updated•4 years ago
|
Comment 4•4 years ago
|
||
On desktop with an android user agent set I can't even scroll this page with a mouse/touchpad scroll. Clicking and dragging reproduces the same bug as on android. Have they implemented their own scrolling behaviour??
Mozregression points to this - bug 1656418.
Partial pre-render is currently only enabled on Nightly. Karl, can you confirm that you can only reproduce this on nightly. And does setting layout.animation.prerender.partial=false
fix it? It does for me.
Scrolling is still janky but it's like that in Chrome and Firefox both with and without webrender. I'm guessing it's due to whatever way they're implementing the scrolling. The profile shows there's no webrender perf issue here.
Comment 5•4 years ago
|
||
The component has been changed since the backlog priority was decided, so we're resetting it.
For more information, please visit auto_nag documentation.
Reporter | ||
Comment 6•4 years ago
|
||
Jamie,
layout.animation.prerender.partial
set to false
on desktop with RDM on desktop and a Firefox Android UA doesn't fix it for me.
but it fixes it on android device indeed.
Updated•2 years ago
|
Comment 7•2 years ago
|
||
Set release status flags based on info from the regressing bug 1656418
:hiro, since you are the author of the regressor, bug 1656418, could you take a look?
For more information, please visit auto_nag documentation.
Updated•2 years ago
|
Comment 8•2 years ago
|
||
The pref causes this has already been disabled.
Updated•2 years ago
|
Comment 9•2 years ago
|
||
Set release status flags based on info from the regressing bug 1656418
Updated•2 years ago
|
Description
•