209494 2020-03-24 14:02:00 -0700 [ Mac wk1 ] fast/events/mouse-cursor-no-mousemove.html is flaky failing. 2020-05-14 04:58:04 -0700 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Mac macOS 10.14 RESOLVED DUPLICATE 211884 https://bugs.webkit.org/show_bug.cgi?id=211884 InRadar P2 Normal --- 1 Lawrence.j graouts graouts graouts koivisto webkit-bot-watchers-bugzilla webkit-bug-importer oldest_to_newest 1633470 0 Lawrence.j 2020-03-24 14:02:00 -0700 fast/events/mouse-cursor-no-mousemove.html Description: This test is flaky failing on Mac wk1. It has been flaky failing throughout the visible history. History: https://results.webkit.org/?suite=layout-tests&test=fast%2Fevents%2Fmouse-cursor-no-mousemove.html&platform=mac&flavor=wk1&limit=50000 Diff: --- /Volumes/Data/slave/catalina-debug-tests-wk1/build/layout-test-results/fast/events/mouse-cursor-no-mousemove-expected.txt +++ /Volumes/Data/slave/catalina-debug-tests-wk1/build/layout-test-results/fast/events/mouse-cursor-no-mousemove-actual.txt @@ -8,7 +8,7 @@ TEST CASE: Mouse idle, change cursor should not fire mousemove event Cursor Info: type=Pointer hotSpot=0,0 -Cursor Info: type=Help hotSpot=0,0 +Cursor Info: type=Pointer hotSpot=0,0 PASS successfullyParsed is true 1633471 1 webkit-bug-importer 2020-03-24 14:02:21 -0700 <rdar://problem/60840602> 1633477 2 Lawrence.j 2020-03-24 14:13:04 -0700 I have marked this test as failing while this issue is investigated. https://trac.webkit.org/changeset/258938/webkit Also, I can reproduce this issue by running the command below. run-webkit-tests --iterations 777 --child-processes 50 --force -f fast/events/mouse-cursor-no-mousemove.html [144/777] fast/events/mouse-cursor-no-mousemove.html failed unexpectedly (text diff) [145/777] fast/events/mouse-cursor-no-mousemove.html failed unexpectedly (text diff) Retrying 1 unexpected failure ... Running 1 WebKitTestRunner. 775 tests ran as expected, 2 didn't: 1652545 3 graouts 2020-05-14 01:38:54 -0700 This test uses two setTimeout() calls with non-zero values, not a big surprise that it's flaky. 1652549 4 graouts 2020-05-14 01:55:07 -0700 Oh dear: // The amount of time to wait for a cursor update on style and layout changes // Set to 50Hz, no need to be faster than common screen refresh rate static const Seconds cursorUpdateInterval { 20_ms }; 1652554 5 graouts 2020-05-14 02:13:11 -0700 Filed bug 211884 to address the 20ms timer. 1652555 6 399341 graouts 2020-05-14 02:16:34 -0700 Created attachment 399341 Patch 1652581 7 graouts 2020-05-14 04:06:14 -0700 Got a patch up for bug 211884 as well. If the bots are pleased, I'll dupe this bug to that one so that we don't use a timer at all and instead reliably can use requestAnimationFrame() in the test. 1652585 8 graouts 2020-05-14 04:58:04 -0700 *** This bug has been marked as a duplicate of bug 211884 *** 399341 2020-05-14 02:16:34 -0700 2020-05-14 02:18:11 -0700 Patch bug-209494-20200514111633.patch text/plain 3093 graouts U3VidmVyc2lvbiBSZXZpc2lvbjogMjYxNjc5CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggM2UzNGFjYTVmZDhhYjQ5NjkzY2I4 MGVjYmQ2MmI5NTRmOTUxMjdiNC4uN2ZhZmJlODgzZjM4ZjQ4MGY2MmM3NGExZTg1ZTU5YmQwODQ4 YTk4OCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDIwLTA1LTE0ICBBbnRvaW5lIFF1aW50ICA8 Z3Jhb3V0c0BhcHBsZS5jb20+CisKKyAgICAgICAgWyBNYWMgd2sxIF0gZmFzdC9ldmVudHMvbW91 c2UtY3Vyc29yLW5vLW1vdXNlbW92ZS5odG1sIGlzIGZsYWt5IGZhaWxpbmcuCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDk0OTQKKyAgICAgICAgPHJk YXI6Ly9wcm9ibGVtLzYwODQwNjAyPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q UyEpLgorCisgICAgICAgIEFkZHJlc3MgdGVzdCBmbGFraW5lc3MgYnkgdXNpbmcgYSBwb2xsaW5n IGFwcHJvYWNoIHJhdGhlciB0aGFuIHdhaXQgYW4gYXJiaXRyYXJ5IGFtb3VudCBvZiB0aW1lLgor ICAgICAgICBJIGZpbGVkIGJ1ZyB3ZWJraXQub3JnL2IvMjExODg0IHRvIGFkZHJlc3MgdGhlIDIw bXMgdGltZXIgZm9yIGN1cnNvciB1cGRhdGVzIGluIEV2ZW50SGFuZGxlci4KKworICAgICAgICAq IGZhc3QvZXZlbnRzL21vdXNlLWN1cnNvci1uby1tb3VzZW1vdmUtZXhwZWN0ZWQudHh0OgorICAg ICAgICAqIGZhc3QvZXZlbnRzL21vdXNlLWN1cnNvci1uby1tb3VzZW1vdmUuaHRtbDoKKwogMjAy MC0wNS0xMyAgTXlsZXMgQy4gTWF4ZmllbGQgIDxtbWF4ZmllbGRAYXBwbGUuY29tPgogCiAgICAg ICAgIGZhc3QvdGV4dC9tdWx0aXBsZS1jb2RldW5pdC12ZXJ0aWNhbC11cHJpZ2h0Lmh0bWwgaXMg ZmFpbGluZwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC9ldmVudHMvbW91c2UtY3Vyc29y LW5vLW1vdXNlbW92ZS1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9mYXN0L2V2ZW50cy9tb3Vz ZS1jdXJzb3Itbm8tbW91c2Vtb3ZlLWV4cGVjdGVkLnR4dAppbmRleCBhNWQwNjQ1ZTliNzYxMTNm N2M0MWVkYWQ5ZjI2Y2ZmZTk0ZjdlNjkxLi4xYmNjYjdjODM1OTE1ZmE0NGUwZjJiMjcyMTA2N2E4 ZTM5N2RjN2YzIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9mYXN0L2V2ZW50cy9tb3VzZS1jdXJz b3Itbm8tbW91c2Vtb3ZlLWV4cGVjdGVkLnR4dAorKysgYi9MYXlvdXRUZXN0cy9mYXN0L2V2ZW50 cy9tb3VzZS1jdXJzb3Itbm8tbW91c2Vtb3ZlLWV4cGVjdGVkLnR4dApAQCAtOCw4ICs4LDcgQEAg QnVnIDg1MzQzCiAKIFRFU1QgQ0FTRTogTW91c2UgaWRsZSwgY2hhbmdlIGN1cnNvciBzaG91bGQg bm90IGZpcmUgbW91c2Vtb3ZlIGV2ZW50CiBDdXJzb3IgSW5mbzogdHlwZT1Qb2ludGVyIGhvdFNw b3Q9MCwwCi1DdXJzb3IgSW5mbzogdHlwZT1IZWxwIGhvdFNwb3Q9MCwwCi0KK1BBU1Mgd2luZG93 LmludGVybmFscy5nZXRDdXJyZW50Q3Vyc29ySW5mbygpIGJlY2FtZSAidHlwZT1IZWxwIGhvdFNw b3Q9MCwwIgogUEFTUyBzdWNjZXNzZnVsbHlQYXJzZWQgaXMgdHJ1ZQogCiBURVNUIENPTVBMRVRF CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0L2V2ZW50cy9tb3VzZS1jdXJzb3Itbm8tbW91 c2Vtb3ZlLmh0bWwgYi9MYXlvdXRUZXN0cy9mYXN0L2V2ZW50cy9tb3VzZS1jdXJzb3Itbm8tbW91 c2Vtb3ZlLmh0bWwKaW5kZXggYzA3MjBhNDllYjE3ZmJlMDg4ODA4NDNkNmExYjI5ZjgzZWJjMDA3 ZC4uODE3NWQwZTVhOWQwOTg3YWZmZmU2NzZmODUwYmFkNjY0NTk4ODMxMSAxMDA2NDQKLS0tIGEv TGF5b3V0VGVzdHMvZmFzdC9ldmVudHMvbW91c2UtY3Vyc29yLW5vLW1vdXNlbW92ZS5odG1sCisr KyBiL0xheW91dFRlc3RzL2Zhc3QvZXZlbnRzL21vdXNlLWN1cnNvci1uby1tb3VzZW1vdmUuaHRt bApAQCAtMTQsNyArMTQsNiBAQAogPGJyLz4KIDxkaXYgaWQ9ImNvbnNvbGUiPjwvZGl2PgogPHNj cmlwdD4KLXZhciBDVVJTT1JfVVBEQVRFX0RFTEFZID0gNTA7CiAKIGRlc2NyaXB0aW9uKCJUZXN0 IHRoYXQgdGhlcmUgaXMgbm8gbW91c2Vtb3ZlIGV2ZW50IGZpcmVkIHdoZW4gY2hhbmdpbmcgY3Vy c29yLiIpOwogCkBAIC0zOSwxNiArMzgsMTQgQEAgaWYgKHdpbmRvdy5ldmVudFNlbmRlcikgewog ICAgICAgICBmaW5pc2hKU1Rlc3QoKTsKICAgICB9KTsKICAgICBub2RlLnN0eWxlLmN1cnNvciA9 ICdoZWxwJzsKLSAgICBzZXRUaW1lb3V0KGZ1bmN0aW9uKCkgewotICAgICAgICBkZWJ1ZygnQ3Vy c29yIEluZm86ICcgKyB3aW5kb3cuaW50ZXJuYWxzLmdldEN1cnJlbnRDdXJzb3JJbmZvKCkpOwot ICAgICAgICBkZWJ1ZygnJyk7Ci0gICAgfSwgQ1VSU09SX1VQREFURV9ERUxBWSk7CiAKLSAgICAv LyBHaXZlIHNvbWUgdGltZSBmb3IgdGhlIGNoYW5nZSB0byByZXNvbHZlLiBGYWtlIG1vdXNlbW92 ZSBldmVudCB0aGF0IGNhdXNlZCBidWcsIGlzIHVzaW5nIGEgdGltZXIKLSAgICBzZXRUaW1lb3V0 KGZ1bmN0aW9uKCkgewotICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgndGVzdC1jb250 YWluZXInKS5zdHlsZS5kaXNwbGF5ID0gJ25vbmUnOwotICAgICAgICBmaW5pc2hKU1Rlc3QoKTsK LSAgICB9LCAxNTApOworICAgIHNob3VsZEJlY29tZUVxdWFsVG9TdHJpbmcoJ3dpbmRvdy5pbnRl cm5hbHMuZ2V0Q3VycmVudEN1cnNvckluZm8oKScsICd0eXBlPUhlbHAgaG90U3BvdD0wLDAnLCAo KSA9PiB7CisgICAgICAgIC8vIEdpdmUgc29tZSB0aW1lIGZvciB0aGUgY2hhbmdlIHRvIHJlc29s dmUuIEZha2UgbW91c2Vtb3ZlIGV2ZW50IHRoYXQgY2F1c2VkIGJ1ZywgaXMgdXNpbmcgYSB0aW1l cgorICAgICAgICBzZXRUaW1lb3V0KGZ1bmN0aW9uKCkgeworICAgICAgICAgICAgZG9jdW1lbnQu Z2V0RWxlbWVudEJ5SWQoJ3Rlc3QtY29udGFpbmVyJykuc3R5bGUuZGlzcGxheSA9ICdub25lJzsK KyAgICAgICAgICAgIGZpbmlzaEpTVGVzdCgpOworICAgICAgICB9LCAxMDApOworICAgIH0pOwog fQogCiA8L3NjcmlwdD4K