207066 2020-01-31 13:13:22 -0800 [ macOS wk1 ] editing/execCommand/insert-nested-lists.html is flaky failing 2026-01-05 10:18:08 -0800 1 1 1 Unclassified WebKit HTML Editing WebKit Nightly Build Unspecified Unspecified RESOLVED WONTFIX InRadar P2 Normal --- 1 jacob_uphoff webkit-unassigned rniwa tsavell webkit-bot-watchers-bugzilla webkit-bug-importer wenson_hsieh oldest_to_newest 1613544 0 jacob_uphoff 2020-01-31 13:13:22 -0800 editing/execCommand/insert-nested-lists.html This flaky failure has been happening for all visible history on Catalina and Mojave I was able to reproduce by running the test in iterations History: https://results.webkit.org/?suite=layout-tests&test=editing%2FexecCommand%2Finsert-nested-lists.html Diff: https://build.webkit.org/results/Apple-Catalina-Debug-WK1-Tests/r255510%20(2186)/editing/execCommand/insert-nested-lists-diff.txt 1613545 1 389398 jacob_uphoff 2020-01-31 13:16:55 -0800 Created attachment 389398 Update Test Expectations 1613551 2 webkit-bug-importer 2020-01-31 13:21:35 -0800 <rdar://problem/59072372> 1613555 3 389398 tsavell 2020-01-31 13:25:15 -0800 Comment on attachment 389398 Update Test Expectations Clearing flags on attachment: 389398 Committed r255520: <https://trac.webkit.org/changeset/255520> 1613771 4 ap 2020-02-01 18:26:06 -0800 Presumably this part is insufficient to deflake it? // By default, AppKit on macOS coalesces undo operations that occur within the same runloop. Wait until the next // runloop before inserting another unordered list to ensure that it gets its own entry in the undo stack. await new Promise((resolve) => { if (window.testRunner) testRunner.runUIScript(`(function() { uiController.uiScriptComplete(); })()`, resolve); else setTimeout(resolve, 500); }); 1637055 5 rniwa 2020-04-02 14:09:54 -0700 (In reply to Alexey Proskuryakov from comment #4) > Presumably this part is insufficient to deflake it? > > // By default, AppKit on macOS coalesces undo operations that occur > within the same runloop. Wait until the next > // runloop before inserting another unordered list to ensure that it > gets its own entry in the undo stack. > await new Promise((resolve) => { > if (window.testRunner) > testRunner.runUIScript(`(function() { > uiController.uiScriptComplete(); })()`, resolve); > else > setTimeout(resolve, 500); > }); Hm... going to the UI script in theory should be sufficient but I guess not. 1637085 6 wenson_hsieh 2020-04-02 14:56:59 -0700 (In reply to Ryosuke Niwa from comment #5) > (In reply to Alexey Proskuryakov from comment #4) > > Presumably this part is insufficient to deflake it? > > > > // By default, AppKit on macOS coalesces undo operations that occur > > within the same runloop. Wait until the next > > // runloop before inserting another unordered list to ensure that it > > gets its own entry in the undo stack. > > await new Promise((resolve) => { > > if (window.testRunner) > > testRunner.runUIScript(`(function() { > > uiController.uiScriptComplete(); })()`, resolve); > > else > > setTimeout(resolve, 500); > > }); > > Hm... going to the UI script in theory should be sufficient but I guess not. I’m not sure if runUIScript is sufficient to ensure that the we proceed after the next runloop in WebKit1. Maybe we need to call uiController.doAsyncTask() instead? 2169349 7 jbedard 2026-01-05 10:18:08 -0800 Closing old WebKitLegacy bugs as we cut-back WebKitLegacy testing. See https://bugs.webkit.org/show_bug.cgi?id=304436. 389398 2020-01-31 13:16:55 -0800 2020-01-31 13:25:15 -0800 Update Test Expectations bug-207066-20200131131654.patch text/plain 1646 jacob_uphoff U3VidmVyc2lvbiBSZXZpc2lvbjogMjU1NTE0CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYjFlNTBhMWQ3ODZmMDJkYTQwMTlm MWVmMjc3YTQ1NjljZGY1N2RiMi4uYmJhMWYxNDZmNjFjYzBlN2ZjNGExYzExZGFmNGQwM2Y2N2U5 MTExNiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEyIEBACisyMDIwLTAxLTMxICBKYWNvYiBVcGhvZmYgIDxq YWNvYl91cGhvZmZAYXBwbGUuY29tPgorCisgICAgICAgIFsgbWFjT1Mgd2sxIF0gZWRpdGluZy9l eGVjQ29tbWFuZC9pbnNlcnQtbmVzdGVkLWxpc3RzLmh0bWwgaXMgZmxha3kgZmFpbGluZworICAg ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjA3MDY2CisKKyAg ICAgICAgVW5yZXZpZXdlZCB0ZXN0IGdhcmRlbmluZy4KKworICAgICAgICAqIHBsYXRmb3JtL21h Yy13azEvVGVzdEV4cGVjdGF0aW9uczoKKwogMjAyMC0wMS0zMSAgSmFzb24gTGF3cmVuY2UgIDxs YXdyZW5jZS5qQGFwcGxlLmNvbT4KIAogICAgICAgICBbIE1vamF2ZSB3azIgUmVsZWFzZSBdIGlt cG9ydGVkL3czYy93ZWItcGxhdGZvcm0tdGVzdHMvaHRtbC93ZWJhcHBhcGlzL3RpbWVycy90eXBl LWxvbmctc2V0aW50ZXJ2YWwuaHRtbCBmbGFreSBmYWlsdXJlCmRpZmYgLS1naXQgYS9MYXlvdXRU ZXN0cy9wbGF0Zm9ybS9tYWMtd2sxL1Rlc3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9wbGF0 Zm9ybS9tYWMtd2sxL1Rlc3RFeHBlY3RhdGlvbnMKaW5kZXggYmYyMmRhZWNmZjViNzFiZTNmNjNj ZDJlNTZiZDYwNWJlMGNlZDkwZC4uMDMzMThjYjgyYWMyYzEzM2Y1YTE4N2RlYTY3MWQ3NTdhMWU4 YWU5ZSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjLXdrMS9UZXN0RXhwZWN0 YXRpb25zCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL21hYy13azEvVGVzdEV4cGVjdGF0aW9u cwpAQCAtODY2LDQgKzg2Niw2IEBAIHdlYmtpdC5vcmcvYi8yMDY5NzQgaHR0cC90ZXN0cy9zZWN1 cml0eS9jb250ZW50U2VjdXJpdHlQb2xpY3kvYmxvY2stYWxsLW1peGVkLWNvCiAKIHdlYmtpdC5v cmcvYi8yMDcwNDYgaW1wb3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy9odG1sL3NlbWFudGlj cy9lbWJlZGRlZC1jb250ZW50L21lZGlhLWVsZW1lbnRzL3NlZWtpbmcvc2Vlay10by1jdXJyZW50 VGltZS5odG1sIFsgUGFzcyBGYWlsdXJlIF0KIAotd2Via2l0Lm9yZy9iLzIwNzA1OSBpbXBvcnRl ZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL0luZGV4ZWREQi9pbnRlcmxlYXZlZC1jdXJzb3JzLXNt YWxsLmh0bWwgWyBQYXNzIEZhaWx1cmUgXQpcIE5vIG5ld2xpbmUgYXQgZW5kIG9mIGZpbGUKK3dl YmtpdC5vcmcvYi8yMDcwNTkgaW1wb3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy9JbmRleGVk REIvaW50ZXJsZWF2ZWQtY3Vyc29ycy1zbWFsbC5odG1sIFsgUGFzcyBGYWlsdXJlIF0KKword2Vi a2l0Lm9yZy9iLzIwNzA2NiBlZGl0aW5nL2V4ZWNDb21tYW5kL2luc2VydC1uZXN0ZWQtbGlzdHMu aHRtbCBbIFBhc3MgRmFpbHVyZSBdClwgTm8gbmV3bGluZSBhdCBlbmQgb2YgZmlsZQo=