217718 2020-10-14 11:08:38 -0700 Scrolls in the passive handler region should be reported as handled promptly 2020-10-18 13:27:33 -0700 1 1 1 Unclassified WebKit Scrolling Safari Technology Preview Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=158439 InRadar P2 Normal --- 1 simon.fraser simon.fraser fujii simon.fraser thorton webkit-bug-importer oldest_to_newest 1697791 0 simon.fraser 2020-10-14 11:08:38 -0700 The WebContent process reports back to the UI process when events are handled (and for wheel events this is used to know if the page overrode history swipes, for example). When handling wheel events in the passive event handler region, we should report that the scrolls are handled promptly to avoid the UI process queueing up events, which impacts scrolling performance. 1697792 1 webkit-bug-importer 2020-10-14 11:09:22 -0700 <rdar://problem/70300104> 1697801 2 411354 simon.fraser 2020-10-14 11:49:53 -0700 Created attachment 411354 Patch 1697963 3 simon.fraser 2020-10-14 16:56:35 -0700 https://bugs.webkit.org/show_bug.cgi?id=217718 1699125 4 fujii 2020-10-18 13:27:33 -0700 Filed: Bug 217850 – [WinCairo] sluggish page scrolling for wheel events since r268499 411354 2020-10-14 11:49:53 -0700 2020-10-14 14:02:27 -0700 Patch bug-217718-20201014114953.patch text/plain 3735 simon.fraser U3VidmVyc2lvbiBSZXZpc2lvbjogMjY4NDc2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IGY0NDZhZDVmYmI1M2RkYWYw ZmM1YjAwYzQzMGYxMDhlNDA0YmI4ODkuLmZiM2Y0ZTZlMDQ1YjY2YzI3ZjU5MjM5OGU0NzQ3ZjM3 YzQwOGZmMDQgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjIgQEAKKzIwMjAtMTAtMTQgIFNpbW9uIEZy YXNlciAgPHNpbW9uLmZyYXNlckBhcHBsZS5jb20+CisKKyAgICAgICAgU2Nyb2xscyBpbiB0aGUg cGFzc2l2ZSBoYW5kbGVyIHJlZ2lvbiBzaG91bGQgYmUgcmVwb3J0ZWQgYXMgaGFuZGxlZCBwcm9t cHRseQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjE3 NzE4CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhl IFVJIHByb2Nlc3Mgd2lsbCBlbnF1ZXVlIHdoZWVsIGV2ZW50cyB0byBzZW5kIGxhdGVyIGlmIHRo ZSBXZWJDb250ZW50IHByb2Nlc3MgZG9lc24ndAorICAgICAgICBwcm9tcHRseSByZXNwb25kIHRo YXQgaXQgaGFzIGhhbmRsZWQgdGhlIHByZXZpb3VzIGV2ZW50LiBXaGVuIHNjcm9sbGluZyBpbiB0 aGUgcGFzc2l2ZQorICAgICAgICBldmVudCByZWdpb24sIHdlIHdhbnQgdGhlIFdlYkNvbnRlbnQg cHJvY2VzcyB0byBhcHBlYXIgcmVzcG9uc2l2ZSwgc28gcmVzcG9uZCB0byB0aGUgVUkKKyAgICAg ICAgcHJvY2VzcyB3aGVuIHdlJ3ZlIHNjcm9sbGVkICh3aGljaCBpcyBiZWZvcmUgdGhlIGV2ZW50 IGhhcyBiZWVuIGRpc3BhdGNoZWQgZm9yIERPTSBldmVudHMpLgorCisgICAgICAgIEhhcmQgdG8g dGVzdCBiZWNhdXNlIENQVS1jb25zdHJhaW5lZCBib3RzIGRvbid0IHNlbmQgd2hlZWwgZXZlbnRz IGluIGEgcHJlZGljdGFibGUgd2F5LgorCisgICAgICAgICogV2ViUHJvY2Vzcy9XZWJQYWdlL0V2 ZW50RGlzcGF0Y2hlci5jcHA6CisgICAgICAgIChXZWJLaXQ6OkV2ZW50RGlzcGF0Y2hlcjo6d2hl ZWxFdmVudCk6CisgICAgICAgICogV2ViUHJvY2Vzcy9XZWJQYWdlL1dlYlBhZ2UuY3BwOgorICAg ICAgICAoV2ViS2l0OjpXZWJQYWdlOjp3aGVlbEV2ZW50KToKKwogMjAyMC0xMC0xNCAgU2VyZ2lv IFZpbGxhciBTZW5pbiAgPHN2aWxsYXJAaWdhbGlhLmNvbT4KIAogICAgICAgICBbR0xJQl0gRml4 IGxpbmtpbmcgZXJyb3IgZm9yIHN5c3RlbXMgbm90IHByb3ZpZGluZyA8c3lzL21lbWZkLmg+CmRp ZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvV2ViUGFnZS9FdmVudERpc3BhdGNo ZXIuY3BwIGIvU291cmNlL1dlYktpdC9XZWJQcm9jZXNzL1dlYlBhZ2UvRXZlbnREaXNwYXRjaGVy LmNwcAppbmRleCAyZWNmOTE0MTQ4OGU5YWFiYWJlZGM1ZDFhMWJiZDBmNTkyZDY2N2NlLi5lNDZj MGM5MDE4NDczMjViMDgwYWFjOWMyYThkMTE5ZjlhMzM4YTZiIDEwMDY0NAotLS0gYS9Tb3VyY2Uv V2ViS2l0L1dlYlByb2Nlc3MvV2ViUGFnZS9FdmVudERpc3BhdGNoZXIuY3BwCisrKyBiL1NvdXJj ZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQYWdlL0V2ZW50RGlzcGF0Y2hlci5jcHAKQEAgLTE0Niwx MiArMTQ2LDEzIEBAIHZvaWQgRXZlbnREaXNwYXRjaGVyOjp3aGVlbEV2ZW50KFBhZ2VJZGVudGlm aWVyIHBhZ2VJRCwgY29uc3QgV2ViV2hlZWxFdmVudCYgd2hlCiAgICAgICAgIFNjcm9sbGluZ1Ro cmVhZDo6ZGlzcGF0Y2goW3Njcm9sbGluZ1RyZWUsIHdoZWVsRXZlbnQsIHBsYXRmb3JtV2hlZWxF dmVudCwgcHJvY2Vzc2luZ1N0ZXBzLCBwYWdlSUQsIHByb3RlY3RlZFRoaXMgPSBtYWtlUmVmKCp0 aGlzKV0gewogICAgICAgICAgICAgYXV0byByZXN1bHQgPSBzY3JvbGxpbmdUcmVlLT5oYW5kbGVX aGVlbEV2ZW50KHBsYXRmb3JtV2hlZWxFdmVudCwgcHJvY2Vzc2luZ1N0ZXBzKTsKIAotICAgICAg ICAgICAgaWYgKHJlc3VsdC5uZWVkc01haW5UaHJlYWRQcm9jZXNzaW5nKCkpIHsKLSAgICAgICAg ICAgICAgICBwcm90ZWN0ZWRUaGlzLT5kaXNwYXRjaFdoZWVsRXZlbnRWaWFNYWluVGhyZWFkKHBh Z2VJRCwgd2hlZWxFdmVudCwgcmVzdWx0LnN0ZXBzKTsKLSAgICAgICAgICAgICAgICByZXR1cm47 Ci0gICAgICAgICAgICB9CisgICAgICAgICAgICBpZiAocmVzdWx0Lm5lZWRzTWFpblRocmVhZFBy b2Nlc3NpbmcoKSkKKyAgICAgICAgICAgICAgICBwcm90ZWN0ZWRUaGlzLT5kaXNwYXRjaFdoZWVs RXZlbnRWaWFNYWluVGhyZWFkKHBhZ2VJRCwgd2hlZWxFdmVudCwgcHJvY2Vzc2luZ1N0ZXBzKTsK IAotICAgICAgICAgICAgcHJvdGVjdGVkVGhpcy0+c2VuZERpZFJlY2VpdmVFdmVudChwYWdlSUQs IHdoZWVsRXZlbnQudHlwZSgpLCByZXN1bHQud2FzSGFuZGxlZCk7CisgICAgICAgICAgICAvLyBJ ZiB3ZSBzY3JvbGxlZCBvbiB0aGUgc2Nyb2xsaW5nIHRocmVhZCAoZXZlbiBpZiB3ZSBzZW5kIHRo ZSBldmVudCB0byB0aGUgbWFpbiB0aHJlYWQgZm9yIHBhc3NpdmUgZXZlbnQgaGFuZGxlcnMpCisg ICAgICAgICAgICAvLyByZXNwb25kIHRvIHRoZSBVSSBwcm9jZXNzIHRoYXQgdGhlIGV2ZW50IHdh cyBoYW5kbGVkLgorICAgICAgICAgICAgaWYgKHJlc3VsdC5zdGVwcy5jb250YWlucyhXaGVlbEV2 ZW50UHJvY2Vzc2luZ1N0ZXBzOjpTY3JvbGxpbmdUaHJlYWQpKQorICAgICAgICAgICAgICAgIHBy b3RlY3RlZFRoaXMtPnNlbmREaWRSZWNlaXZlRXZlbnQocGFnZUlELCB3aGVlbEV2ZW50LnR5cGUo KSwgcmVzdWx0Lndhc0hhbmRsZWQpOwogICAgICAgICB9KTsKIAogICAgICAgICByZXR1cm4gcHJv Y2Vzc2luZ1N0ZXBzOwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9XZWJQcm9jZXNzL1dlYlBh Z2UvV2ViUGFnZS5jcHAgYi9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJQYWdl LmNwcAppbmRleCAwNDYwYzZkNzYzMzBiODliZjMxNzhhN2VjZDc2NDA4OTRmYWYxYWQ4Li40NjI2 NzQ5MDhhNzhlOTk4ZDkyMmJiZWZlYTg0Mzc3YTQ2NTM0MzJkIDEwMDY0NAotLS0gYS9Tb3VyY2Uv V2ViS2l0L1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJQYWdlLmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0 L1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJQYWdlLmNwcApAQCAtMjg3Miw3ICsyODcyLDggQEAgdm9p ZCBXZWJQYWdlOjp3aGVlbEV2ZW50KGNvbnN0IFdlYldoZWVsRXZlbnQmIHdoZWVsRXZlbnQsIE9w dGlvblNldDxXaGVlbEV2ZW50UHIKIAogICAgIGJvb2wgaGFuZGxlZCA9IGhhbmRsZVdoZWVsRXZl bnQod2hlZWxFdmVudCwgbV9wYWdlLmdldCgpLCBwcm9jZXNzaW5nU3RlcHMpOwogCi0gICAgc2Vu ZChNZXNzYWdlczo6V2ViUGFnZVByb3h5OjpEaWRSZWNlaXZlRXZlbnQoc3RhdGljX2Nhc3Q8dWlu dDMyX3Q+KHdoZWVsRXZlbnQudHlwZSgpKSwgaGFuZGxlZCkpOworICAgIGlmIChwcm9jZXNzaW5n U3RlcHMuY29udGFpbnMoV2hlZWxFdmVudFByb2Nlc3NpbmdTdGVwczo6TWFpblRocmVhZEZvclNj cm9sbGluZykpCisgICAgICAgIHNlbmQoTWVzc2FnZXM6OldlYlBhZ2VQcm94eTo6RGlkUmVjZWl2 ZUV2ZW50KHN0YXRpY19jYXN0PHVpbnQzMl90Pih3aGVlbEV2ZW50LnR5cGUoKSksIGhhbmRsZWQp KTsKIH0KIAogc3RhdGljIGJvb2wgaGFuZGxlS2V5RXZlbnQoY29uc3QgV2ViS2V5Ym9hcmRFdmVu dCYga2V5Ym9hcmRFdmVudCwgUGFnZSogcGFnZSkK