224454 2021-04-12 14:37:54 -0700 REGRESSION (r275792): [ iOS wk2 ] TestWebKitAPI.IndexedDB.DatabaseProcessKill is a flakey timeout 2021-04-13 09:54:48 -0700 1 1 1 Unclassified WebKit WebKit API WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 jenner simon.fraser beidson cdumez sihui_liu simon.fraser webkit-bot-watchers-bugzilla webkit-bug-importer oldest_to_newest 1749389 0 jenner 2021-04-12 14:37:54 -0700 TestWebKitAPI.IndexedDB.DatabaseProcessKill is a flakey timeout on iOS wk2. HISTORY: https://results.webkit.org/?suite=api-tests&test=TestWebKitAPI.IndexedDB.DatabaseProcessKill Link to stdio file where timeout is noted: https://build.webkit.org/#/builders/28/builds/910/steps/11/logs/stdio It is flakey, but it does timeout more than it passes. 1749391 1 webkit-bug-importer 2021-04-12 14:38:11 -0700 <rdar://problem/76558602> 1749422 2 jenner 2021-04-12 16:01:06 -0700 This appears to be trivially reproducible with a release build, but does not appear to affect debug or production builds. As such, I did reproduce the failure at iOS Release ToT using the following test: run-api-tests --ios-simulator TestWebKitAPI.IndexedDB.DatabaseProcessKill --child-process=50 Note that it is only occurring on release, and will pass on Production and Debug builds. I was also able to verify a regress point. The timeouts occur in r275792, and did not occur in 275791. It appears this changeset caused the timeouts: https://trac.webkit.org/changeset/275792/webkit 1749511 3 425825 simon.fraser 2021-04-12 20:12:54 -0700 Created attachment 425825 Patch 1749533 4 425825 sihui_liu 2021-04-12 22:08:41 -0700 Comment on attachment 425825 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=425825&action=review > Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBDatabaseProcessKill.mm:85 > [webView _test_waitForDidFinishNavigation]; Why not just replace this with TestWebKitAPI::Util::run(&receivedScriptMessage)? 1749695 5 425825 simon.fraser 2021-04-13 08:57:39 -0700 Comment on attachment 425825 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=425825&action=review >> Tools/TestWebKitAPI/Tests/WebKitCocoa/IndexedDBDatabaseProcessKill.mm:85 >> [webView _test_waitForDidFinishNavigation]; > > Why not just replace this with TestWebKitAPI::Util::run(&receivedScriptMessage)? Maybe it could just be removed. I'll test. 1749714 6 simon.fraser 2021-04-13 09:54:48 -0700 https://trac.webkit.org/changeset/275890/webkit 425825 2021-04-12 20:12:54 -0700 2021-04-12 22:49:23 -0700 Patch bug-224454-20210412201253.patch text/plain 5308 simon.fraser U3VidmVyc2lvbiBSZXZpc2lvbjogMjc1ODI2CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggYzA0MDEwMTIyYjY2YmE0YjYzMTU0MGQxMTJiNWI0ODRm ODE3ZGMyYi4uNmYxMTE2YmUyMGY0OTNjM2RmZjA0MmU1NGY1OGVhYzNlMDhmYTJlNCAxMDA2NDQK LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIy IEBACisyMDIxLTA0LTEyICBTaW1vbiBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgor CisgICAgICAgIFJFR1JFU1NJT04gKHIyNzU3OTIpOiBbIGlPUyB3azIgXSBUZXN0V2ViS2l0QVBJ LkluZGV4ZWREQi5EYXRhYmFzZVByb2Nlc3NLaWxsIGlzIGEgZmxha2V5IHRpbWVvdXQKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIyNDQ1NAorCisgICAg ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgdGVzdCB3YXMg cmFjZXk7IHRoZSBjb250ZW50IHJhbiBzY3JpcHQgaW1tZWRpYXRlbHksIGFuZCBpdCB3YXMgcG9z c2libGUgZm9yCisgICAgICAgIHRoZSBmaXJzdCBzZXQgb2YgbWVzc2FnZXMgdG8gYmUgcmVjZWl2 ZWQgYmVmb3JlIC1fdGVzdF93YWl0Rm9yRGlkRmluaXNoTmF2aWdhdGlvbgorICAgICAgICByZXR1 cm5lZDsgdGhlIHRlc3Qgd291bGQgdGhlbiBzdGFsbCBiZWNhdXNlIG5vdGhpbmcgdHJpZ2dlcmVk IGFueSBtb3JlIG1lc3NhZ2VzLgorCisgICAgICAgIEZpeCBieSBoYXZpbmcgdGhlIGNvbnRlbnQg cnVuIHRoZSB0ZXN0cyBmcm9tIGEgZnVuY3Rpb24sIGFuZCBjYWxsIHRoYXQgZnVuY3Rpb24KKyAg ICAgICAgZnJvbSB0aGUgdGVzdC4gQWxzbyBoYW5kbGUgdGhlIGNhc2Ugd2hlcmUgcmVjZWl2ZWRT Y3JpcHRNZXNzYWdlIGNhbiBiZSB0cnVlCisgICAgICAgIGJlZm9yZSBlbnRlcmluZyB0aGUgZmly c3QgVGVzdFdlYktpdEFQSTo6VXRpbDo6cnVuKCkuCisKKyAgICAgICAgKiBUZXN0V2ViS2l0QVBJ L1Rlc3RzL1dlYktpdENvY29hL0luZGV4ZWREQkRhdGFiYXNlUHJvY2Vzc0tpbGwtMS5odG1sOgor ICAgICAgICAqIFRlc3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Q29jb2EvSW5kZXhlZERCRGF0YWJh c2VQcm9jZXNzS2lsbC5tbToKKyAgICAgICAgKFRFU1QpOgorCiAyMDIxLTA0LTExICBDb21taXQg UXVldWUgIDxjb21taXQtcXVldWVAd2Via2l0Lm9yZz4KIAogICAgICAgICBVbnJldmlld2VkLCBy ZXZlcnRpbmcgcjI3NTgwMS4KZGlmZiAtLWdpdCBhL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMv V2ViS2l0Q29jb2EvSW5kZXhlZERCRGF0YWJhc2VQcm9jZXNzS2lsbC0xLmh0bWwgYi9Ub29scy9U ZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL0luZGV4ZWREQkRhdGFiYXNlUHJvY2Vzc0tp bGwtMS5odG1sCmluZGV4IGFiNDI5OWJkODcyMWU0N2Y5ODYwMTdmNGJkZTkxYjQ2MjhlMDU1Y2Qu LmU1NThjNzQ2MGNjNjJhMDEzMDhlOWQwYjkxOWFkNGE3MzlmZWI2MzcgMTAwNjQ0Ci0tLSBhL1Rv b2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Q29jb2EvSW5kZXhlZERCRGF0YWJhc2VQcm9j ZXNzS2lsbC0xLmh0bWwKKysrIGIvVG9vbHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXRDb2Nv YS9JbmRleGVkREJEYXRhYmFzZVByb2Nlc3NLaWxsLTEuaHRtbApAQCAtMSwzMyArMSwzNSBAQAog PHNjcmlwdD4KIAotdmFyIGRhdGFiYXNlOworZnVuY3Rpb24gcnVuVGVzdCgpCit7CisgICAgdmFy IGRhdGFiYXNlOwogCi1mb3IgKHZhciBpID0gMDsgaSA8IDEwMDsgKytpKSB7Ci0gICAgdmFyIHJl cXVlc3QgPSB3aW5kb3cuaW5kZXhlZERCLmRlbGV0ZURhdGFiYXNlKCJJbmRleGVkREJEYXRhYmFz ZVByb2Nlc3NLaWxsIik7Ci0gICAgcmVxdWVzdC5vbnN1Y2Nlc3MgPSBmdW5jdGlvbigpIHsKLSAg ICAgICAgd2luZG93LndlYmtpdC5tZXNzYWdlSGFuZGxlcnMudGVzdEhhbmRsZXIucG9zdE1lc3Nh Z2UoJ0RlbGV0ZVJlcXVlc3REb25lJyk7Ci0gICAgfQotICAgIHJlcXVlc3Qub25lcnJvciA9IGZ1 bmN0aW9uKCkgewotICAgICAgICB3aW5kb3cud2Via2l0Lm1lc3NhZ2VIYW5kbGVycy50ZXN0SGFu ZGxlci5wb3N0TWVzc2FnZSgnRGVsZXRlUmVxdWVzdEVycm9yJyk7Ci0gICAgfQorICAgIGZvciAo dmFyIGkgPSAwOyBpIDwgMTAwOyArK2kpIHsKKyAgICAgICAgdmFyIHJlcXVlc3QgPSB3aW5kb3cu aW5kZXhlZERCLmRlbGV0ZURhdGFiYXNlKCJJbmRleGVkREJEYXRhYmFzZVByb2Nlc3NLaWxsIik7 CisgICAgICAgIHJlcXVlc3Qub25zdWNjZXNzID0gZnVuY3Rpb24oKSB7CisgICAgICAgICAgICB3 aW5kb3cud2Via2l0Lm1lc3NhZ2VIYW5kbGVycy50ZXN0SGFuZGxlci5wb3N0TWVzc2FnZSgnRGVs ZXRlUmVxdWVzdERvbmUnKTsKKyAgICAgICAgfQorICAgICAgICByZXF1ZXN0Lm9uZXJyb3IgPSBm dW5jdGlvbigpIHsKKyAgICAgICAgICAgIHdpbmRvdy53ZWJraXQubWVzc2FnZUhhbmRsZXJzLnRl c3RIYW5kbGVyLnBvc3RNZXNzYWdlKCdEZWxldGVSZXF1ZXN0RXJyb3InKTsKKyAgICAgICAgfQog Ci0gICAgcmVxdWVzdCA9IHdpbmRvdy5pbmRleGVkREIub3BlbigiSW5kZXhlZERCRGF0YWJhc2VQ cm9jZXNzS2lsbCIpOwotICAgIHJlcXVlc3Qub25zdWNjZXNzID0gZnVuY3Rpb24oKSB7Ci0gICAg ICAgIHdpbmRvdy53ZWJraXQubWVzc2FnZUhhbmRsZXJzLnRlc3RIYW5kbGVyLnBvc3RNZXNzYWdl KCdPcGVuUmVxdWVzdERvbmUnKTsKLSAgICB9Ci0gICAgcmVxdWVzdC5vbmVycm9yID0gZnVuY3Rp b24oKSB7Ci0gICAgICAgIHdpbmRvdy53ZWJraXQubWVzc2FnZUhhbmRsZXJzLnRlc3RIYW5kbGVy LnBvc3RNZXNzYWdlKCdPcGVuUmVxdWVzdEVycm9yJyk7Ci0gICAgfQotICAgIHJlcXVlc3Qub251 cGdyYWRlbmVlZGVkID0gZnVuY3Rpb24oZSkgewotICAgICAgICBkYXRhYmFzZSA9IGUudGFyZ2V0 LnJlc3VsdDsKLSAgICAgICAgZGF0YWJhc2Uub25lcnJvciA9IGZ1bmN0aW9uKCkgewotICAgICAg ICAgICAgbm90aWZ5RGF0YWJhc2VFcnJvclJlY2VpdmVkID0gZnVuY3Rpb24oKSB7Ci0gICAgICAg ICAgICAgICAgd2luZG93LndlYmtpdC5tZXNzYWdlSGFuZGxlcnMudGVzdEhhbmRsZXIucG9zdE1l c3NhZ2UoJ0RhdGFiYXNlRXJyb3JSZWNlaXZlZCcpOworICAgICAgICByZXF1ZXN0ID0gd2luZG93 LmluZGV4ZWREQi5vcGVuKCJJbmRleGVkREJEYXRhYmFzZVByb2Nlc3NLaWxsIik7CisgICAgICAg IHJlcXVlc3Qub25zdWNjZXNzID0gZnVuY3Rpb24oKSB7CisgICAgICAgICAgICB3aW5kb3cud2Vi a2l0Lm1lc3NhZ2VIYW5kbGVycy50ZXN0SGFuZGxlci5wb3N0TWVzc2FnZSgnT3BlblJlcXVlc3RE b25lJyk7CisgICAgICAgIH0KKyAgICAgICAgcmVxdWVzdC5vbmVycm9yID0gZnVuY3Rpb24oKSB7 CisgICAgICAgICAgICB3aW5kb3cud2Via2l0Lm1lc3NhZ2VIYW5kbGVycy50ZXN0SGFuZGxlci5w b3N0TWVzc2FnZSgnT3BlblJlcXVlc3RFcnJvcicpOworICAgICAgICB9CisgICAgICAgIHJlcXVl c3Qub251cGdyYWRlbmVlZGVkID0gZnVuY3Rpb24oZSkgeworICAgICAgICAgICAgZGF0YWJhc2Ug PSBlLnRhcmdldC5yZXN1bHQ7CisgICAgICAgICAgICBkYXRhYmFzZS5vbmVycm9yID0gZnVuY3Rp b24oKSB7CisgICAgICAgICAgICAgICAgbm90aWZ5RGF0YWJhc2VFcnJvclJlY2VpdmVkID0gZnVu Y3Rpb24oKSB7CisgICAgICAgICAgICAgICAgICAgIHdpbmRvdy53ZWJraXQubWVzc2FnZUhhbmRs ZXJzLnRlc3RIYW5kbGVyLnBvc3RNZXNzYWdlKCdEYXRhYmFzZUVycm9yUmVjZWl2ZWQnKTsKKyAg ICAgICAgICAgICAgICB9CisgICAgICAgICAgICAgICAgc2V0VGltZW91dCgic2V0VGltZW91dChu b3RpZnlEYXRhYmFzZUVycm9yUmVjZWl2ZWQsIDApIiwgMCk7CiAgICAgICAgICAgICB9Ci0gICAg ICAgICAgICBzZXRUaW1lb3V0KCJzZXRUaW1lb3V0KG5vdGlmeURhdGFiYXNlRXJyb3JSZWNlaXZl ZCwgMCkiLCAwKTsKKyAgICAgICAgICAgIHdpbmRvdy53ZWJraXQubWVzc2FnZUhhbmRsZXJzLnRl c3RIYW5kbGVyLnBvc3RNZXNzYWdlKCdPcGVuUmVxdWVzdFVwZ3JhZGVOZWVkZWQnKTsKICAgICAg ICAgfQotICAgICAgICB3aW5kb3cud2Via2l0Lm1lc3NhZ2VIYW5kbGVycy50ZXN0SGFuZGxlci5w b3N0TWVzc2FnZSgnT3BlblJlcXVlc3RVcGdyYWRlTmVlZGVkJyk7CiAgICAgfQogfQotCiA8L3Nj cmlwdD4KZGlmZiAtLWdpdCBhL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Q29jb2Ev SW5kZXhlZERCRGF0YWJhc2VQcm9jZXNzS2lsbC5tbSBiL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVz dHMvV2ViS2l0Q29jb2EvSW5kZXhlZERCRGF0YWJhc2VQcm9jZXNzS2lsbC5tbQppbmRleCA3MzA3 N2VjZGZkMzY1YjUyOTA1M2Q0N2NiYTJhMTRlZDA1MWVkNmVhLi40NDZhODdiYmIzMGFlN2U5NmNh ZTc5MWViZDM2ZjQ4MmJlYzIxYjNhIDEwMDY0NAotLS0gYS9Ub29scy9UZXN0V2ViS2l0QVBJL1Rl c3RzL1dlYktpdENvY29hL0luZGV4ZWREQkRhdGFiYXNlUHJvY2Vzc0tpbGwubW0KKysrIGIvVG9v bHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXRDb2NvYS9JbmRleGVkREJEYXRhYmFzZVByb2Nl c3NLaWxsLm1tCkBAIC04NCwxMyArODQsMTYgQEAgVEVTVChJbmRleGVkREIsIERhdGFiYXNlUHJv Y2Vzc0tpbGwpCiAgICAgW3dlYlZpZXcgbG9hZFJlcXVlc3Q6cmVxdWVzdF07CiAgICAgW3dlYlZp ZXcgX3Rlc3Rfd2FpdEZvckRpZEZpbmlzaE5hdmlnYXRpb25dOwogCisgICAgcmVjZWl2ZWRTY3Jp cHRNZXNzYWdlID0gZmFsc2U7CisgICAgW3dlYlZpZXcgZXZhbHVhdGVKYXZhU2NyaXB0OkAicnVu VGVzdCgpOyIgY29tcGxldGlvbkhhbmRsZXI6bmlsXTsKKwogICAgIGJvb2wga2lsbGVkREJQcm9j ZXNzID0gZmFsc2U7CiAgICAgd2hpbGUgKHRydWUpIHsKICAgICAgICAgaWYgKGRhdGFiYXNlRXJy b3JSZWNlaXZlZCkKICAgICAgICAgICAgIGJyZWFrOwogCi0gICAgICAgIHJlY2VpdmVkU2NyaXB0 TWVzc2FnZSA9IGZhbHNlOwogICAgICAgICBUZXN0V2ViS2l0QVBJOjpVdGlsOjpydW4oJnJlY2Vp dmVkU2NyaXB0TWVzc2FnZSk7CisgICAgICAgIHJlY2VpdmVkU2NyaXB0TWVzc2FnZSA9IGZhbHNl OwogICAgICAgICBpZiAoIWtpbGxlZERCUHJvY2VzcyAmJiBvcGVuUmVxdWVzdFVwZ3JhZGVOZWVk ZWQpIHsKICAgICAgICAgICAgIGtpbGxlZERCUHJvY2VzcyA9IHRydWU7CiAgICAgICAgICAgICBb Y29uZmlndXJhdGlvbi5nZXQoKS53ZWJzaXRlRGF0YVN0b3JlIF90ZXJtaW5hdGVOZXR3b3JrUHJv Y2Vzc107Cg==