188894 2018-08-23 11:33:28 -0700 [Cocoa] First scroll gesture in pinned, non-rubber-banding WKWebView may fail to initiate back/forward swipe 2018-08-23 12:51:40 -0700 1 1 1 Unclassified WebKit WebKit2 Other Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 mitz mitz simon.fraser thorton oldest_to_newest 1453141 0 mitz 2018-08-23 11:33:28 -0700 Sometimes the first swipe gesture in a WKWebView would fail to initiate a back/forward swipe despite the view being pinned to the edge and non-rubber-banding. This happens because of a race condition in EventDispatcher::wheelEvent, where setting the rubber-band state is dispatched to the scrolling thread, whereas querying the rubber-banding state happens immediately right after that on the event dispatcher thread, under tryToHandleWheelEvent(). 1453145 1 mitz 2018-08-23 11:38:42 -0700 <rdar://problem/43651434> 1453146 2 347941 mitz 2018-08-23 11:42:37 -0700 Created attachment 347941 Set the rubber-band state from the event dispatcher thread 1453171 3 mitz 2018-08-23 12:51:40 -0700 Fixed in <https://trac.webkit.org/r235236>. 347941 2018-08-23 11:42:37 -0700 2018-08-23 11:46:11 -0700 Set the rubber-band state from the event dispatcher thread bug-188894-20180823114236.patch text/plain 2207 mitz SW5kZXg6IFNvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJL aXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAyMzUyMzEpCisrKyBTb3VyY2UvV2ViS2l0L0NoYW5nZUxv Zwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDE4LTA4LTIzICBEYW4gQmVybnN0 ZWluICA8bWl0ekBhcHBsZS5jb20+CisKKyAgICAgICAgW0NvY29hXSBGaXJzdCBzY3JvbGwgZ2Vz dHVyZSBpbiBwaW5uZWQsIG5vbi1ydWJiZXItYmFuZGluZyBXS1dlYlZpZXcgbWF5IGZhaWwgdG8g aW5pdGlhdGUgYmFjay9mb3J3YXJkIHN3aXBlCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQu b3JnL3Nob3dfYnVnLmNnaT9pZD0xODg4OTQKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzQzNjUx NDM0PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICog V2ViUHJvY2Vzcy9XZWJQYWdlL0V2ZW50RGlzcGF0Y2hlci5jcHA6CisgICAgICAgIChXZWJLaXQ6 OkV2ZW50RGlzcGF0Y2hlcjo6d2hlZWxFdmVudCk6IFNldCB0aGUgcnViYmVyLWJhbmQgc3RhdGUg b24gdGhlIFNjcm9sbGluZ1RyZWUKKyAgICAgICAgICBzeW5jaHJvbm91c2x5IHJhdGhlciB0aGFu IGRpc3BhdGNoaW5nIGRvaW5nIHRoYXQgdG8gdGhlIHNjcm9sbGluZyB0aHJlYWQuIFRoaXMgaXMg c2FmZSB0bworICAgICAgICAgIGRvIGJlY2F1c2UgU2Nyb2xsaW5nVHJlZSBzeW5jaHJub2l6ZXMg YWNjZXNzIHRvIHRoZSBydWJiZXItYmFuZCBzdGF0ZSB3aXRoIGFuIGludGVybmFsCisgICAgICAg ICAgbXV0ZXguCisKIDIwMTgtMDgtMjMgIENvbW1pdCBRdWV1ZSAgPGNvbW1pdC1xdWV1ZUB3ZWJr aXQub3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIHJvbGxpbmcgb3V0IHIyMzUyMTYuCkluZGV4 OiBTb3VyY2UvV2ViS2l0L1dlYlByb2Nlc3MvV2ViUGFnZS9FdmVudERpc3BhdGNoZXIuY3BwCj09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQYWdlL0V2ZW50RGlzcGF0 Y2hlci5jcHAJKHJldmlzaW9uIDIzNTE0MykKKysrIFNvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9X ZWJQYWdlL0V2ZW50RGlzcGF0Y2hlci5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEyMCwxMSArMTIw LDggQEAgdm9pZCBFdmVudERpc3BhdGNoZXI6OndoZWVsRXZlbnQodWludDY0XwogICAgICAgICAv LyBXZWJDb3JlIHNob3VsZCBhbHdheXMga25vdyB0aGUgY3VycmVudCBzdGF0ZSBhbmQga25vdyB3 aGVuIGl0IGNoYW5nZXMgc28gdGhlCiAgICAgICAgIC8vIHNjcm9sbGluZyB0cmVlIGNhbiBiZSBu b3RpZmllZC4KICAgICAgICAgLy8gV2Ugb25seSBuZWVkIHRvIGRvIHRoaXMgYXQgdGhlIGJlZ2lu bmluZyBvZiB0aGUgZ2VzdHVyZS4KLSAgICAgICAgaWYgKHBsYXRmb3JtV2hlZWxFdmVudC5waGFz ZSgpID09IFBsYXRmb3JtV2hlZWxFdmVudFBoYXNlQmVnYW4pIHsKLSAgICAgICAgICAgIFNjcm9s bGluZ1RocmVhZDo6ZGlzcGF0Y2goW3Njcm9sbGluZ1RyZWUsIGNhblJ1YmJlckJhbmRBdExlZnQs IGNhblJ1YmJlckJhbmRBdFJpZ2h0LCBjYW5SdWJiZXJCYW5kQXRUb3AsIGNhblJ1YmJlckJhbmRB dEJvdHRvbV0gewotICAgICAgICAgICAgICAgIHNjcm9sbGluZ1RyZWUtPnNldENhblJ1YmJlckJh bmRTdGF0ZShjYW5SdWJiZXJCYW5kQXRMZWZ0LCBjYW5SdWJiZXJCYW5kQXRSaWdodCwgY2FuUnVi YmVyQmFuZEF0VG9wLCBjYW5SdWJiZXJCYW5kQXRCb3R0b20pOwotICAgICAgICAgICAgfSk7Ci0g ICAgICAgIH0KKyAgICAgICAgaWYgKHBsYXRmb3JtV2hlZWxFdmVudC5waGFzZSgpID09IFBsYXRm b3JtV2hlZWxFdmVudFBoYXNlQmVnYW4pCisgICAgICAgICAgICBzY3JvbGxpbmdUcmVlLT5zZXRD YW5SdWJiZXJCYW5kU3RhdGUoY2FuUnViYmVyQmFuZEF0TGVmdCwgY2FuUnViYmVyQmFuZEF0Umln aHQsIGNhblJ1YmJlckJhbmRBdFRvcCwgY2FuUnViYmVyQmFuZEF0Qm90dG9tKTsKIAogICAgICAg ICBTY3JvbGxpbmdUcmVlOjpFdmVudFJlc3VsdCByZXN1bHQgPSBzY3JvbGxpbmdUcmVlLT50cnlU b0hhbmRsZVdoZWVsRXZlbnQocGxhdGZvcm1XaGVlbEV2ZW50KTsKIAo=