Bug 233187

Summary: [GTK][WPE] displayDidRefresh can shortcut waking up the scrolling thread
Product: WebKit Reporter: Chris Lord <clord>
Component: ScrollingAssignee: Chris Lord <clord>
Status: RESOLVED FIXED    
Severity: Normal CC: alex, cmarcelo, ews-watchlist, fred.wang, jamesr, luiz, simon.fraser, tonikitoo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Chris Lord
Reported 2021-11-16 05:05:59 PST
There's an #ifdef'd out bit of code in ThreadedScrollingTree::displayDidRefresh that will shortcut waking up the scrolling thread if there are no recently handled wheel events or active animations. This seems like an easy win - the code within the #ifdef has changed and perhaps didn't previously fully apply for us, but now does, so we should remove the #ifdef.
Attachments
Patch (1.58 KB, patch)
2021-11-16 05:10 PST, Chris Lord 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Lord
Comment 1 2021-11-16 05:10:05 PST
Created attachment 444371 [details] Patch
EWS
Comment 2 2021-11-16 05:40:59 PST
Committed r285858 (244284@main): <https://commits.webkit.org/244284@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 444371 [details].
Radar WebKit Bug Importer
Comment 3 2021-11-16 05:41:24 PST
<rdar://problem/85454805>
Note You need to log in before you can comment on or make changes to this bug.