Bug 219776 – [GTK][WPE] Async overflow scrolling is disabled by default due to brokenness

• Status: RESOLVED
• Resolution: FIXED
• Component: Scrolling

[Chris Lord]

Sorry for the slightly vague title - Currently, AsyncOverflowScrollingEnabled is disabled in WebPreferencesInternal for Nicosia because enabling it results in broken rendering on scrolled overflow scrolling layers and their descendents. This is obviously less than ideal, scrolling is often not done on the root frame and disabling async overflow scrolling causes us to fall back to both main-thread scrolling and results in more painting and worse-feeling scrolling.

The bug appears to be boundsOrigin on layers not being respected correctly. We should fix this and enable this setting - I have a patch that fixes at least the most obvious issues. I'm not certain it's a complete fix, but we need wider testing to identify further issues.

In the future, it'd probably be ideal for there to be more shared code here between Mac and Linux, where Mac has a complete and likely more bug-free implementation. I think it'd be good to do some consolidation/refactoring and renaming in this code path - though this patch is quite small, it took a long time to figure out.

[Chris Lord]

Created attachment 415986 [details]
Patch

[Chris Lord]

*** Bug 203412 has been marked as a duplicate of this bug. ***

[Chris Lord]

Created attachment 416000 [details]
Patch

[EWS]

Committed r270683: <https://trac.webkit.org/changeset/270683>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 416000 [details].

[Radar WebKit Bug Importer]

<rdar://problem/72226809>