233187 2021-11-16 05:05:59 -0800 [GTK][WPE] displayDidRefresh can shortcut waking up the scrolling thread 2021-11-16 05:41:24 -0800 1 1 1 Unclassified WebKit Scrolling WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 clord clord alex cmarcelo ews-watchlist fred.wang jamesr luiz simon.fraser tonikitoo webkit-bug-importer oldest_to_newest 1815441 0 clord 2021-11-16 05:05:59 -0800 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. 1815442 1 444371 clord 2021-11-16 05:10:05 -0800 Created attachment 444371 Patch 1815443 2 ews-feeder 2021-11-16 05:40:59 -0800 Committed r285858 (244284@main): <https://commits.webkit.org/244284@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 444371. 1815444 3 webkit-bug-importer 2021-11-16 05:41:24 -0800 <rdar://problem/85454805> 444371 2021-11-16 05:10:05 -0800 2021-11-16 05:41:00 -0800 Patch bug-233187-20211116131004.patch text/plain 1616 clord U3VidmVyc2lvbiBSZXZpc2lvbjogMjg1Mzk2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYTRkNTEwMzVlOGRmYmMx MmFhYmVkMWI3MDMxZjU5Yjk1NTViNDc2Mi4uMjA0ZDU0NmJiYTc2YWNjNTk3YTQ1NjE0NzJjMzUz ZjQ3YWU2ODg3ZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDIxLTExLTE2ICBDaHJp cyBMb3JkICA8Y2xvcmRAaWdhbGlhLmNvbT4KKworICAgICAgICBbR1RLXVtXUEVdIGRpc3BsYXlE aWRSZWZyZXNoIGNhbiBzaG9ydGN1dCB3YWtpbmcgdXAgdGhlIHNjcm9sbGluZyB0aHJlYWQKKyAg ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzMzE4NworCisg ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFsbG93IHNjcm9s bGluZyB0aHJlYWQgd2FrZS11cCBzaG9ydGN1dCBvbiBHVEsgYW5kIFdQRSAobWF0Y2hpbmcKKyAg ICAgICAgYmVoYXZpb3VyIG9uIGV2ZXJ5IG90aGVyIHBsYXRmb3JtKS4KKworICAgICAgICBObyBu ZXcgdGVzdHMsIGNvdmVyZWQgYnkgZXhpc3RpbmcgdGVzdHMuCisKKyAgICAgICAgKiBwYWdlL3Nj cm9sbGluZy9UaHJlYWRlZFNjcm9sbGluZ1RyZWUuY3BwOgorICAgICAgICAoV2ViQ29yZTo6VGhy ZWFkZWRTY3JvbGxpbmdUcmVlOjpkaXNwbGF5RGlkUmVmcmVzaCk6CisKIDIwMjEtMTEtMTUgIENo cmlzIExvcmQgIDxjbG9yZEBpZ2FsaWEuY29tPgogCiAgICAgICAgIFtHTElCXSB0d2l0Y2gudHYg Zm9yY2VzIHN5bmNocm9ub3VzIHNjcm9sbGluZwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUv cGFnZS9zY3JvbGxpbmcvVGhyZWFkZWRTY3JvbGxpbmdUcmVlLmNwcCBiL1NvdXJjZS9XZWJDb3Jl L3BhZ2Uvc2Nyb2xsaW5nL1RocmVhZGVkU2Nyb2xsaW5nVHJlZS5jcHAKaW5kZXggOWVjMTAyNzZm NWRiNWMxMDE5YThkZmFjOGNhYTNlZjg3OWQzOTYzZS4uNzI4ODhhNDY0NzJlZTgzZTY4N2M1Yjcw YzllOWQ5ZDlhZTg4NTI1NyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGFnZS9zY3JvbGxp bmcvVGhyZWFkZWRTY3JvbGxpbmdUcmVlLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wYWdlL3Nj cm9sbGluZy9UaHJlYWRlZFNjcm9sbGluZ1RyZWUuY3BwCkBAIC00ODgsMTAgKzQ4OCw4IEBAIHZv aWQgVGhyZWFkZWRTY3JvbGxpbmdUcmVlOjpkaXNwbGF5RGlkUmVmcmVzaChQbGF0Zm9ybURpc3Bs YXlJRCBkaXNwbGF5SUQpCiAgICAgaWYgKGRpc3BsYXlJRCAhPSB0aGlzLT5kaXNwbGF5SUQoKSkK ICAgICAgICAgcmV0dXJuOwogCi0jaWYgIVBMQVRGT1JNKFdQRSkgJiYgIVBMQVRGT1JNKEdUSykK ICAgICBpZiAoIXNjcm9sbGluZ1RocmVhZElzQWN0aXZlKQogICAgICAgICByZXR1cm47Ci0jZW5k aWYKIAogICAgIFNjcm9sbGluZ1RocmVhZDo6ZGlzcGF0Y2goW3Byb3RlY3RlZFRoaXMgPSBSZWYg eyAqdGhpcyB9XSgpIHsKICAgICAgICAgcHJvdGVjdGVkVGhpcy0+ZGlzcGxheURpZFJlZnJlc2hP blNjcm9sbGluZ1RocmVhZCgpOwo=