251312 2023-01-27 22:47:55 -0800 [ iOS BigSur+ ] fast/loader/stateobjects/popstate-does-not-fire-with-page-cache.html is a flaky failure. 2023-03-16 14:54:28 -0700 1 1 1 Unclassified WebKit New Bugs Other Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 bharris9 cdumez cdumez webkit-bot-watchers-bugzilla webkit-bug-importer oldest_to_newest 1929147 0 bharris9 2023-01-27 22:47:55 -0800 fast/loader/stateobjects/popstate-does-not-fire-with-page-cache.html is a flaky failure on iOS 16, BigSur and later platforms. HISTORY: https://results.webkit.org/?suite=layout-tests&test=fast%2Floader%2Fstateobjects%2Fpopstate-does-not-fire-with-page-cache.html&limit=10000 Crash Log is attached. CRASH LOG TEXT: @@ -10,8 +10,7 @@ pageshow fired - persisted=false PASS testWindowPopstateFireCount is 0 going back to page 1 in initial state -popstate fired with state null -PASS testWindowPopstateFireCount is 1 +FAIL testWindowPopstateFireCount should be 1. Was 0. PASS successfullyParsed is true 1929148 1 webkit-bug-importer 2023-01-27 22:48:17 -0800 <rdar://problem/104775065> 1929149 2 bharris9 2023-01-27 22:54:49 -0800 I was able to reproduce this failure on iOS 16 Release ToT as follows: run-webkit-test --ios-simulator fast/loader/stateobjects/popstate-does-not-fire-with-page-cache.html --iteration 100 With the iterations, I was able to reproduce 22 failures. I was able to reproduce this failure on Ventura Release ToT as follows: run-webkit-test fast/loader/stateobjects/popstate-does-not-fire-with-page-cache.html --iteration 100 With the iterations, I was able to reproduce 4 failures of 100. 1929152 3 464699 bharris9 2023-01-27 22:56:07 -0800 Created attachment 464699 text diff 1929155 4 bharris9 2023-01-27 23:06:28 -0800 Pull request: https://github.com/WebKit/WebKit/pull/9279 1929433 5 ews-feeder 2023-01-30 08:52:04 -0800 Test gardening commit 259561@main (df3d1454d73d): <https://commits.webkit.org/259561@main> Reviewed commits have been landed. Closing PR #9279 and removing active labels. 1929488 6 bharris9 2023-01-30 12:16:03 -0800 Pull request: https://github.com/WebKit/WebKit/pull/9334 1929497 7 ews-feeder 2023-01-30 12:25:24 -0800 Test gardening commit 259581@main (bda96a710d73): <https://commits.webkit.org/259581@main> Reviewed commits have been landed. Closing PR #9334 and removing active labels. 1929592 8 cdumez 2023-01-30 15:50:42 -0800 Pull request: https://github.com/WebKit/WebKit/pull/9355 1929925 9 ews-feeder 2023-01-31 16:45:21 -0800 Committed 259649@main (67c6a041f39c): <https://commits.webkit.org/259649@main> Reviewed commits have been landed. Closing PR #9355 and removing active labels. 1930594 10 bharris9 2023-02-02 12:01:42 -0800 Due to test still failing, reopening. Will mark expectations for macOS while this is being investigated. 1930597 11 bharris9 2023-02-02 12:15:31 -0800 Pull request: https://github.com/WebKit/WebKit/pull/9548 1930602 12 ews-feeder 2023-02-02 12:35:30 -0800 Test gardening commit 259774@main (c3714cf7be45): <https://commits.webkit.org/259774@main> Reviewed commits have been landed. Closing PR #9548 and removing active labels. 1931034 13 cdumez 2023-02-03 16:11:42 -0800 This is the part that is flaky: ``` function onTestWindowPageShow(event) { debug("pageshow fired - persisted=" + event.persisted); setTimeout(() => { // Should not have fired a popstate event since the navigation wasn't within the same document. shouldBe("testWindowPopstateFireCount", "0"); debug('going back to page 1 in initial state'); setTimeout(function() {testWindow.history.back();}, 0); }, 10); } ``` It seems sometimes `testWindow.history.back()` doesn't cause a back navigation here and the 10ms delay is what makes it flaky. If we don't use a 10ms delay, then it consistently fails. 1931055 14 cdumez 2023-02-03 16:56:59 -0800 Pull request: https://github.com/WebKit/WebKit/pull/9637 1931235 15 ews-feeder 2023-02-05 15:03:19 -0800 Committed 259873@main (9b11ab322031): <https://commits.webkit.org/259873@main> Reviewed commits have been landed. Closing PR #9637 and removing active labels. 1941963 16 ews-feeder 2023-03-16 14:54:28 -0700 Committed 259548.431@safari-7615-branch (d4a83e5b2618): <https://commits.webkit.org/259548.431@safari-7615-branch> Reviewed commits have been landed. Closing PR #468 and removing active labels. 464699 2023-01-27 22:56:07 -0800 2023-01-30 12:37:27 -0800 text diff 251312 - popstate-does-not-fire-with-page-cache-diff.txt text/plain 641 bharris9 LS0tIC9Wb2x1bWVzL0RhdGEvd29ya2VyL0FwcGxlLWlPUy0xNi1TaW11bGF0b3ItUmVsZWFzZS1X SzItVGVzdHMvYnVpbGQvbGF5b3V0LXRlc3QtcmVzdWx0cy9mYXN0L2xvYWRlci9zdGF0ZW9iamVj dHMvcG9wc3RhdGUtZG9lcy1ub3QtZmlyZS13aXRoLXBhZ2UtY2FjaGUtZXhwZWN0ZWQudHh0Cisr KyAvVm9sdW1lcy9EYXRhL3dvcmtlci9BcHBsZS1pT1MtMTYtU2ltdWxhdG9yLVJlbGVhc2UtV0sy LVRlc3RzL2J1aWxkL2xheW91dC10ZXN0LXJlc3VsdHMvZmFzdC9sb2FkZXIvc3RhdGVvYmplY3Rz L3BvcHN0YXRlLWRvZXMtbm90LWZpcmUtd2l0aC1wYWdlLWNhY2hlLWFjdHVhbC50eHQKQEAgLTEw LDggKzEwLDcgQEAKIHBhZ2VzaG93IGZpcmVkIC0gcGVyc2lzdGVkPWZhbHNlCiBQQVNTIHRlc3RX aW5kb3dQb3BzdGF0ZUZpcmVDb3VudCBpcyAwCiBnb2luZyBiYWNrIHRvIHBhZ2UgMSBpbiBpbml0 aWFsIHN0YXRlCi1wb3BzdGF0ZSBmaXJlZCB3aXRoIHN0YXRlIG51bGwKLVBBU1MgdGVzdFdpbmRv d1BvcHN0YXRlRmlyZUNvdW50IGlzIDEKK0ZBSUwgdGVzdFdpbmRvd1BvcHN0YXRlRmlyZUNvdW50 IHNob3VsZCBiZSAxLiBXYXMgMC4KIFBBU1Mgc3VjY2Vzc2Z1bGx5UGFyc2VkIGlzIHRydWUKIAog VEVTVCBDT01QTEVURQo=