16976 2008-01-22 11:51:28 -0800 caret position needs to be set *before* firing/handling input events 2022-07-30 13:18:15 -0700 1 1 1 Unclassified WebKit HTML Editing 528+ (Nightly build) All All RESOLVED CONFIGURATION CHANGED InRadar P2 Normal --- 1 timmmaaayyy webkit-unassigned ahmad.saleem792 ap aroben bfulgham ddkilzer harrison justin.garcia rniwa oldest_to_newest 67962 0 timmmaaayyy 2008-01-22 11:51:28 -0800 Editor::appliedEditing currently dispatches input events and after sets the selection (caret) to the end of the text. This means when handling an input event, you cannot modify the selection without being 'trumped' by this later setting by appliedEditing. I tried making a patch to fix this by simply moving one line in this method, dispatchEditibleContentChangedEvents, 5 lines down so it occurs after the call to setSelection. I created a layout test to verify (see attached). However, the patch causes two layout tests to fail: fast/forms/drag-into-textarea.html and fast/forms/drag-out-of-textarea.html. The output differs in these two tests because rather than cut/pasting, the drag an drop results in both fields being empty or both fields containing the text, respectively. The expected output is correct and I don't think it should be changed, so this attempt to fix breaks text drag and drop, but I'm not sure why. 67963 1 18602 timmmaaayyy 2008-01-22 11:55:45 -0800 Created attachment 18602 test that caret selection is set before firing input event currently fails on recent c.o @ r29678 67966 2 eric 2008-01-22 12:30:23 -0800 The test passes in FF and Opera, and does not actually execute cleanly in IE. 72058 3 aroben 2008-02-26 15:05:23 -0800 <rdar://problem/5766949> 1887865 4 ahmad.saleem792 2022-07-30 09:45:00 -0700 I am unable to reproduce this bug in Safari 15.6 on macOS 12.5 using attached test case and as soon as I type something in <input> field, I get "test passed" and it is same across all other browsers (Firefox Nightly 105 and Chrome Canary 106). I think it got fixed along the way and this can be marked as "RESOLVED CONFIGURATION CHANGED". Thanks! 18602 2008-01-22 11:55:45 -0800 2008-01-22 11:55:45 -0800 test that caret selection is set before firing input event set-selection-before-input-event.html text/html 1128 timmmaaayyy PGh0bWw+DQoJPGhlYWQ+DQoJCTxzY3JpcHQ+DQoJCQlmdW5jdGlvbiB0ZXN0KCkgew0KCQkJICBk b2N1bWVudC5nZXRFbGVtZW50QnlJZCgidGFyZ2V0IikuYWRkRXZlbnRMaXN0ZW5lcigiaW5wdXQi LCBpbnB1dEhhbmRsZXIsIGZhbHNlKTsNCg0KCQkgICAgICAgIH0NCiAgICAgICAgICAgICAgICAg ICAgICAgIHZhciBmaW5pc2hlZCA9IGZhbHNlOw0KCQkJZnVuY3Rpb24gZmluaXNoKCkgew0KCQkg ICAgICAgICAgaWYgKGZpbmlzaGVkKQ0KICAJCQkgICAgcmV0dXJuOw0KICAgICAgICAgICAgICAg ICAgICAgICAgICBmaW5pc2hlZCA9IHRydWU7DQoJCQkgIGlmIChzZWxlY3Rpb25TdGFydCA9PSAx KQ0KCQkJICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJyZXN1bHQiKS5pbm5lckhUTUwgPSBk b2N1bWVudC5nZXRFbGVtZW50QnlJZCgicmVzdWx0IikuaW5uZXJIVE1MICsgInRlc3QgcGFzc2Vk IjsNCgkJCSAgZWxzZQ0KCQkJICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJyZXN1bHQiKS5p bm5lckhUTUwgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgicmVzdWx0IikuaW5uZXJIVE1MICsg InRlc3QgZmFpbGVkIjsNCgkJCX0NCgkJCXZhciBzZWxlY3Rpb25TdGFydCA9IDA7DQoJCQlmdW5j dGlvbiBpbnB1dEhhbmRsZXIoKSB7DQogIAkJCSAgc2VsZWN0aW9uU3RhcnQgPSBkb2N1bWVudC5n ZXRFbGVtZW50QnlJZCgidGFyZ2V0Iikuc2VsZWN0aW9uU3RhcnQ7DQoJCQkgIGZpbmlzaCgpOw0K CQkgICAgICAgfQ0KCSAgICAgICA8L3NjcmlwdD4NCiAgICAgICA8L2hlYWQ+DQogICAgICAgPGJv ZHkgb25sb2FkID0gInRlc3QoKSI+DQoJICAgICAgIDxwPlRoaXMgdGVzdCBjaGVja3MgdGhhdCB0 aGUgY2FyZXQgaXMgbW92ZWQgdG8gdGhlIGVuZCBvZiB0aGUgaW5wdXQgdGV4dCB0aGF0IGp1c3Qg dHJpZ2dlcmVkIGFuIGlucHV0RXZlbnQgQkVGT1JFIGRpc3BhdGNoaW5nIHRoZSBldmVudCB0byBo YW5kbGVycy48L3A+DQoJICAgICAgIDxpbnB1dCBpZD0idGFyZ2V0IiB0eXBlPSJ0ZXh0IiBvbmlu cHV0PSJpbnB1dEhhbmRsZXIoKSI+PC9pbnB1dD4NCgkgICAgICAgPGRpdiBpZD0icmVzdWx0Ij48 YnI+PC9kaXY+DQogICAgICAgPC9ib2R5Pg0KICAgICAgIDwvaHRtbD4NCg0K