(sw-wr) Stuttery scrolling on https://www.foxitsoftware.com/pdf-reader/
Categories
(Core :: Graphics: WebRender, enhancement)
Tracking
()
People
(Reporter: mayankleoboy1, Assigned: gw)
References
(Blocks 1 open bug)
Details
enable sw-wr
Go to https://www.foxitsoftware.com/pdf-reader/
Scroll on the page, specially the top of the page with the orange background
Reporter | ||
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Matt, looks like a lot of time spent in UnmapTile in D3D11 compositor...
Comment 2•4 years ago
|
||
Indeed there is a lot of time spent there, but as is common with Mayank's profiles, most of the time there appears to just be blocking within the kernel (NtQueryLicenseValue).
My understanding is that this mostly means that other work is slowing down the kernel and it's not necessarily this call that's actually the problem. Nothing else really stands out from the profile though, and than that it's doing a lot of work for advanced blend modes.
Let's chat about this more during the sw-wr meeting tomorrow!
Comment 3•4 years ago
|
||
Looks like picture caching isn't working well for this and we're drawing more than we might need to be. Adding ni?gw to dig into this a bit more.
Assignee | ||
Comment 4•4 years ago
|
||
I can see what's causing the constant invalidation.
The page content contains a mix-blend container right at the start of the content display list, because the fixed position background image has overlay
mix-blend-mode set. In combination with the scrolling content and other fixed position elements, WR currently places all of this content in the same slice to ensure the mix-blend-mode works correctly.
I can think of a couple of possible solutions to this, but I will need to investigate a bit more. It's also possible that the way the current blend container APIs work in WR is making this seem harder than it needs to be.
Assignee | ||
Comment 5•3 years ago
|
||
It seems like this is resolved now from a quick test of the page, either by some recent WR work or by the page content itself changing. Does this seem fine in nightly for you?
Reporter | ||
Comment 7•3 years ago
|
||
Profile from the original page (from webarchive): https://share.firefox.dev/35XJzA6
Profile from the current version of the page: https://share.firefox.dev/3CByjpn
Edit: it is definitely improved. Lets close this bug for now, and file new one if needed.
Reporter | ||
Updated•3 years ago
|
Comment 8•3 years ago
|
||
(In reply to Glenn Watson [:gw] from comment #5)
It seems like this is resolved now from a quick test of the page, either by some recent WR work or by the page content itself changing. Does this seem fine in nightly for you?
Shouldn't the resolution here be WORKSFORME? It is normally FIXED only if there is a patch in the bug or a demonstrated connection to another bug report that has a patch.
Reporter | ||
Updated•3 years ago
|
Description
•