181193 2017-12-30 11:47:19 -0800 REGRESSION (r225494): Mouse pointer reappears shortly after hiding when scrolling using arrow keys 2017-12-31 12:45:23 -0800 1 1 1 Unclassified WebKit WebKit Misc. WebKit Local Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=180247 InRadar, Regression P1 Normal --- 1 mitz mitz ap dino graouts simon.fraser thorton oldest_to_newest 1384982 0 mitz 2017-12-30 11:47:19 -0800 To reproduce this bug, make this window shorter than the webpage, place the mouse pointer somewhere on the page, and press the up or down arrow key to scroll. The mouse pointer hides, then the page scrolls, then shortly afterwards the mouse pointer shows again, which shouldn’t happen. This happens in TOT but not in Safari 11.0.2 (604.4.7.1.3). 1384985 1 mitz 2017-12-30 13:37:34 -0800 Caused by <https://trac.webkit.org/r225494>, the fix for bug 180247. It’s not clear (and the change log doesn’t explain) why cancelAutoHideCursorTimer was made to show the cursor. 1384987 2 mitz 2017-12-30 13:40:07 -0800 rdar://problem/36242565 1384988 3 330262 mitz 2017-12-30 13:56:59 -0800 Created attachment 330262 Removed code to explicitly unhide the cursor when canceling the autohide timer 1384992 4 mitz 2017-12-30 20:39:04 -0800 Thanks, Alexey! Committed <https://trac.webkit.org/r226313>. 1385002 5 graouts 2017-12-31 02:19:04 -0800 Is https://bugs.webkit.org/show_bug.cgi?id=180247 still fixed after this patch? 1385005 6 mitz 2017-12-31 12:45:23 -0800 (In reply to Antoine Quint from comment #5) > Is https://bugs.webkit.org/show_bug.cgi?id=180247 still fixed after this > patch? Before this patch, the mouse pointer would unhide when hitting Esc. After this patch, the mouse pointer remains hidden after hitting Esc, until the mouse is moved. This matches the behavior in QuickTime Player. So I’d say that bug 180247 remains fixed, because the pointer isn’t hidden permanently as it used to be. 330262 2017-12-30 13:56:59 -0800 2017-12-30 20:35:18 -0800 Removed code to explicitly unhide the cursor when canceling the autohide timer bug-181193-20171230135658.patch text/plain 1289 mitz SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDIyNjMxMikKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE0IEBACisyMDE3LTEyLTMwICBEYW4gQmVy bnN0ZWluICA8bWl0ekBhcHBsZS5jb20+CisKKyAgICAgICAgUkVHUkVTU0lPTiAocjIyNTQ5NCk6 IE1vdXNlIHBvaW50ZXIgcmVhcHBlYXJzIHNob3J0bHkgYWZ0ZXIgaGlkaW5nIHdoZW4gc2Nyb2xs aW5nIHVzaW5nIGFycm93IGtleXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv d19idWcuY2dpP2lkPTE4MTE5MworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp LgorCisgICAgICAgICogcGFnZS9FdmVudEhhbmRsZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6 RXZlbnRIYW5kbGVyOjpjYW5jZWxBdXRvSGlkZUN1cnNvclRpbWVyKTogUmVtb3ZlZCBhIGNhbGwg dG8gdW5oaWRlIHRoZSBjdXJzb3IKKyAgICAgICAgICBoZXJlLgorCiAyMDE3LTEyLTMwICBXZW5z b24gSHNpZWggIDx3ZW5zb25faHNpZWhAYXBwbGUuY29tPgogCiAgICAgICAgIFtBdHRhY2htZW50 IFN1cHBvcnRdIFJlbW92ZSBjdXJyZW50IG1hY09TIHN1cHBvcnQgZm9yIGRyYWdnaW5nIGZpbGUt YmFja2VkIGF0dGFjaG1lbnRzCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxl ci5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuY3Bw CShyZXZpc2lvbiAyMjYzMTIpCisrKyBTb3VyY2UvV2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxlci5j cHAJKHdvcmtpbmcgY29weSkKQEAgLTE1ODksOSArMTU4OSw2IEBAIHZvaWQgRXZlbnRIYW5kbGVy OjpjYW5jZWxBdXRvSGlkZUN1cnNvclQKIHsKICAgICBpZiAobV9hdXRvSGlkZUN1cnNvclRpbWVy LmlzQWN0aXZlKCkpCiAgICAgICAgIG1fYXV0b0hpZGVDdXJzb3JUaW1lci5zdG9wKCk7Ci0KLSAg ICBpZiAoYXV0byBwYWdlID0gbV9mcmFtZS5wYWdlKCkpCi0gICAgICAgIHBhZ2UtPmNocm9tZSgp LnNldEN1cnNvckhpZGRlblVudGlsTW91c2VNb3ZlcyhmYWxzZSk7CiB9CiAKIHZvaWQgRXZlbnRI YW5kbGVyOjphdXRvSGlkZUN1cnNvclRpbWVyRmlyZWQoKQo=