Summary: | scrollingTreeNodeDidScroll() should just trigger a rendering udpate | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||
Component: | Scrolling | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | alex, cmarcelo, ews-watchlist, fred.wang, jamesr, luiz, simon.fraser, thorton, tonikitoo, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Simon Fraser (smfr)
2021-04-09 15:05:29 PDT
Created attachment 425660 [details]
Patch
Committed r275790 (236361@main): <https://commits.webkit.org/236361@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 425660 [details]. This patch broke the async scrolling for gtk and wpe, we do not have the call to synchronizeStateFromScrollingTree in Nicosia implementation. I could fix adding a willStartRenderingUpdate implementation to Nicosia that syncs but I'll take this failure as an opportunitty to review the whole synchronization because maybe we have other differences that can cause trouble in the future. Created bug 224824 to solve the regression in WPE and GTK. |