206454 2020-01-17 15:41:02 -0800 REGRESSION(r253636): [GTK] Mouse cursor changes using onMouseXYZ are erratic 2026-01-14 14:39:27 -0800 1 1 1 Unclassified WebKit WebKitGTK Other Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=305509 P2 Normal --- 1 lmoura webkit-unassigned akkimlove88 andersca bugs-noreply cgarcia thorton oldest_to_newest 1608250 0 lmoura 2020-01-17 15:41:02 -0800 After r253636 (after bisecting) the two tests below started failing. * fast/events/mouse-cursor-change.html: ``` @@ -10,13 +10,13 @@ Cursor Info: type=Hand hotSpot=0,0 Mouse down +Cursor Info: type=Hand hotSpot=0,0 + +Mouse hold down, move Cursor Info: type=Progress hotSpot=0,0 -Mouse hold down, move -Cursor Info: type=Hand hotSpot=0,0 - Mouse up -Cursor Info: type=Help hotSpot=0,0 +Cursor Info: type=Progress hotSpot=0,0 PASS successfullyParsed is true ``` * fast/events/mouse-cursor-no-mousemove.html: ``` @@ -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 ``` Testing the first one in MiniBrowserGtk, it seems like the cursor keeps stuck to the style assigned in `onMouseMove`, with the other styles appearing briefly in a erratic manner. Manual testing worked fine in Firefox and Epiphany 3.34 (WebKitGTK 2.26.2). 1612883 1 389233 cgarcia 2020-01-30 01:00:51 -0800 Created attachment 389233 Patch 1612884 2 389233 thorton 2020-01-30 01:15:50 -0800 Comment on attachment 389233 Patch Apologies! 1612923 3 cgarcia 2020-01-30 05:27:53 -0800 Committed r255421: <https://trac.webkit.org/changeset/255421> 389233 2020-01-30 01:00:51 -0800 2020-01-30 01:15:50 -0800 Patch wk2-cursors.diff text/plain 2651 cgarcia ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCAzNTNjYTNjZmVjYS4uZmQ4OGY2OTE0YTIgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3Rz L0NoYW5nZUxvZworKysgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMiBAQAor MjAyMC0wMS0zMCAgQ2FybG9zIEdhcmNpYSBDYW1wb3MgIDxjZ2FyY2lhQGlnYWxpYS5jb20+CisK KyAgICAgICAgUkVHUkVTU0lPTihyMjUzNjM2KTogW0dUS10gTW91c2UgY3Vyc29yIGNoYW5nZXMg dXNpbmcgb25Nb3VzZVhZWiBhcmUgZXJyYXRpYworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjA2NDU0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ndGsvVGVzdEV4cGVjdGF0aW9uczoKKwog MjAyMC0wMS0yOSAgeW91ZW5uIGZhYmxldCAgPHlvdWVubkBhcHBsZS5jb20+CiAKICAgICAgICAg QmxvYiBtZWRpYSBsb2FkaW5nIGRvZXMgbm90IHdvcmsgd2l0aCBHUFVQcm9jZXNzCmRpZmYgLS1n aXQgYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9ndGsvVGVzdEV4cGVjdGF0aW9ucyBiL0xheW91dFRl c3RzL3BsYXRmb3JtL2d0ay9UZXN0RXhwZWN0YXRpb25zCmluZGV4IDE1ZDg0NDU3ZWQwLi5jNGE2 M2U5NzMwMCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Rlc3RFeHBlY3Rh dGlvbnMKKysrIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vZ3RrL1Rlc3RFeHBlY3RhdGlvbnMKQEAg LTEyNCw5ICsxMjQsNiBAQCB3ZWJraXQub3JnL2IvOTg5MjUgZmFzdC92aWV3cG9ydC92aWV3cG9y dC1sZWdhY3kteGh0bWxtcC5odG1sIFsgU2tpcCBdCiAjIFdlYktpdEdUSysgdXNlcyBkaWZmZXJl bnQga2V5Ym9hcmQgYWNjZWxlcmF0b3JzIHRoYW4gU2FmYXJpICh2aWEgQ2hyb21pdW0pLgogZmFz dC9ldmVudHMvb3B0aW9uLXRhYi5odG1sIFsgV29udEZpeCBGYWlsdXJlIF0KIAotd2Via2l0Lm9y Zy9iLzIwNjQ1NCBmYXN0L2V2ZW50cy9tb3VzZS1jdXJzb3ItY2hhbmdlLmh0bWwgWyBGYWlsdXJl IF0KLXdlYmtpdC5vcmcvYi8yMDY0NTQgZmFzdC9ldmVudHMvbW91c2UtY3Vyc29yLW5vLW1vdXNl bW92ZS5odG1sIFsgRmFpbHVyZSBdCi0KICMgVGhlc2UgdGVzdHMgZXhwZWN0IGEgVElGRiBkZWNv ZGVyICh2aWEgQ2hyb21pdW0pLgogIyBUaGVzZSBwYXNzIHdoaWxlIHRoZXkncmUgbWFya2VkIGFz IFdvbnRGaXggLSBjaGVjayBpdC4KIGZhc3QvaW1hZ2VzL2VtYmVkLWltYWdlLmh0bWwgWyBXb250 Rml4IFBhc3MgXQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9DaGFuZ2VMb2cgYi9Tb3VyY2Uv V2ViS2l0L0NoYW5nZUxvZwppbmRleCA5ZmVjMmRkM2FjNi4uOGUwZjEwZGM4ZGUgMTAwNjQ0Ci0t LSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9n CkBAIC0xLDMgKzEsMTQgQEAKKzIwMjAtMDEtMzAgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dh cmNpYUBpZ2FsaWEuY29tPgorCisgICAgICAgIFJFR1JFU1NJT04ocjI1MzYzNik6IFtHVEtdIE1v dXNlIGN1cnNvciBjaGFuZ2VzIHVzaW5nIG9uTW91c2VYWVogYXJlIGVycmF0aWMKKyAgICAgICAg aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIwNjQ1NAorCisgICAgICAg IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFNpbmNlIHIyNTM2MzYgb25s eSBwbGF0Zm9ybXMgZGVmaW5pbmcgSEFWRV9OU0NVUlNPUiBjbGFpbSB0byBzdXBwb3J0IHNldHRp bmcgdGhlIGN1cnNvci4KKworICAgICAgICAqIFdlYlByb2Nlc3MvV2ViQ29yZVN1cHBvcnQvV2Vi Q2hyb21lQ2xpZW50Lmg6IERvIG5vdCBpbXBsZW1lbnQgc3VwcG9ydHNTZXR0aW5nQ3Vyc29yKCkg Zm9yIEdUSyBwb3J0IGVpdGhlci4KKwogMjAyMC0wMS0yOSAgQ2FybG9zIEdhcmNpYSBDYW1wb3Mg IDxjZ2FyY2lhQGlnYWxpYS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZC4gTW92ZSB1bnNldGVu diB0byBXZWJQcm9jZXNzTWFpbi5jcHAgYWZ0ZXIgcjI1NTM0MgpkaWZmIC0tZ2l0IGEvU291cmNl L1dlYktpdC9XZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkNocm9tZUNsaWVudC5oIGIvU291 cmNlL1dlYktpdC9XZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkNocm9tZUNsaWVudC5oCmlu ZGV4IDlhMDNmN2ZkOTFkLi45ZjBmYmI5YzYxNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9X ZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkNocm9tZUNsaWVudC5oCisrKyBiL1NvdXJjZS9X ZWJLaXQvV2ViUHJvY2Vzcy9XZWJDb3JlU3VwcG9ydC9XZWJDaHJvbWVDbGllbnQuaApAQCAtMTk1 LDcgKzE5NSw3IEBAIHByaXZhdGU6CiAKICAgICB2b2lkIHNldEN1cnNvcihjb25zdCBXZWJDb3Jl OjpDdXJzb3ImKSBmaW5hbDsKICAgICB2b2lkIHNldEN1cnNvckhpZGRlblVudGlsTW91c2VNb3Zl cyhib29sKSBmaW5hbDsKLSNpZiAhSEFWRShOU0NVUlNPUikKKyNpZiAhSEFWRShOU0NVUlNPUikg JiYgIVBMQVRGT1JNKEdUSykKICAgICBib29sIHN1cHBvcnRzU2V0dGluZ0N1cnNvcigpIGZpbmFs IHsgcmV0dXJuIGZhbHNlOyB9CiAjZW5kaWYKIAo=