222422 2021-02-25 10:01:53 -0800 REGRESSION (r273461): [ macOS ] imported/w3c/web-platform-tests/media-source/mediasource-duration.html is timing out 2021-02-26 14:14:12 -0800 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=222462 InRadar P2 Normal --- 1 jenner webkit-unassigned tsavell webkit-bot-watchers-bugzilla webkit-bug-importer oldest_to_newest 1733373 0 jenner 2021-02-25 10:01:53 -0800 imported/w3c/web-platform-tests/media-source/mediasource-duration.html is timing out in macOS Catalina and Big Sur, in release and Debug, for Wk1 and Wk2. HISTORY URL: https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fmedia-source%2Fmediasource-duration.html DIFF URL: https://build.webkit.org/results/Apple-BigSur-Release-WK2-Tests/r273484%20(510)/imported/w3c/web-platform-tests/media-source/mediasource-duration-pretty-diff.html DIFF TEXT: --- /Volumes/Data/slave/bigsur-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/media-source/mediasource-duration-expected.txt +++ /Volumes/Data/slave/bigsur-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/media-source/mediasource-duration-actual.txt @@ -1,11 +1,5 @@ +#PID UNRESPONSIVE - WebKitTestRunner (pid 27787) +FAIL: Timed out waiting for notifyDone to be called -FAIL Test seek starts on duration truncation below currentTime The object is in an invalid state. -FAIL Test appendBuffer completes previous seek to truncated duration The object is in an invalid state. -FAIL Test endOfStream completes previous seek to truncated duration The object is in an invalid state. -PASS Test setting same duration multiple times does not fire duplicate durationchange -PASS Test setting the duration to less than the highest starting presentation timestamp will throw -PASS Truncating the duration throws an InvalidStateError exception when new duration is less than the highest buffered range start time of one of the track buffers -PASS Truncating the duration throws an InvalidStateError exception when new duration is less than a buffered coded frame presentation time -PASS Increasing the duration does not trigger any SourceBuffer update -PASS Increasing the duration during media playback does not trigger any SourceBuffer update - +#EOF +#EOF 1733374 1 webkit-bug-importer 2021-02-25 10:02:19 -0800 <rdar://problem/74748843> 1733378 2 jenner 2021-02-25 10:06:20 -0800 Timeouts started at changeset r273461: CHANGESET URL: https://trac.webkit.org/changeset/273461/webkit Changes appear related to the test that is timing out. At this time we cannot reproduce due to issues with the builder bots. The builder bots went down yesterday, and there hasn't been new builds available since yesterday at 1PM. Will test when bots come back up. 1733464 3 jenner 2021-02-25 12:39:46 -0800 This test has been constantly failing for a very long time. The test expectations for it, were set to expect a failure since r206264 CHANGESET URL: https://trac.webkit.org/changeset/206264/webkit Starting at r273461, the test started to timeout. At this point, I'm not certain if we just need to update test expectations again, revert changes made at r273461, or modify the test. 1733487 4 jenner 2021-02-25 13:41:22 -0800 Moving forward we are going to treat this as a regression. I was able to reproduce the timeout at tip of tree using the following test: run-webkit-test imported/w3c/web-platform-tests/media-source/mediasource-duration.html --iterations 100 -f Now, the tricky thing about this one, is that it has been failing for a while, but the test expectations were marked so it passed on the dashboard. So we need to decide if we revert the changes at r273461, or try and fix this test. 1733529 5 421568 jenner 2021-02-25 14:45:08 -0800 Created attachment 421568 Patch 1733530 6 421568 tsavell 2021-02-25 14:47:11 -0800 Comment on attachment 421568 Patch Clearing flags on attachment: 421568 Committed r273514 (234589@main): <https://commits.webkit.org/234589@main> 1733531 7 jenner 2021-02-25 14:47:41 -0800 Updated test expectations to include Slow to determine if changes made in r273461 are a regression, or if it made the test run better. Before the change the test would fail in 21 seconds. Now, it times out at 60 seconds. Changes made may have fixed part of this test, and that is what we are looking to determine. 1733971 8 tsavell 2021-02-26 14:14:04 -0800 It looks like the work to fix this test is happening in https://bugs.webkit.org/show_bug.cgi?id=222462 421568 2021-02-25 14:45:08 -0800 2021-02-25 14:47:11 -0800 Patch bug-222422-20210225144507.patch text/plain 2227 jenner U3VidmVyc2lvbiBSZXZpc2lvbjogMjczNTEzCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYjQyNDMxMTI0YzJjNjQxNzc1OWE3 MGI0OGZmMjAwYTc0ZGVmMTkyZS4uM2U4MTJlZDllMzY2N2JkYTc4YmY5NGI4YjY1NmM1MjBhODA2 MjA4YiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEyIEBACisyMDIxLTAyLTI1ICBSb2JlcnQgSmVubmVyICA8 amVubmVyQGFwcGxlLmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OIChyMjczNDYxKTogWyBtYWNP UyBdIGltcG9ydGVkL3czYy93ZWItcGxhdGZvcm0tdGVzdHMvbWVkaWEtc291cmNlL21lZGlhc291 cmNlLWR1cmF0aW9uLmh0bWwgaXMgdGltaW5nIG91dAorICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjIyNDIyCisKKyAgICAgICAgVW5yZXZpZXdlZCB0ZXN0 IGdhcmRlbmluZy4KKworICAgICAgICAqIHBsYXRmb3JtL21hYy9UZXN0RXhwZWN0YXRpb25zOiBV cGRhdGluZyB0ZXN0IGV4cGVjdGF0aW9ucyB0byBpbmNsdWRlIHNsb3cgdG8gZGV0ZXJtaW5lIGlm IGNoYW5nZXMgY2F1c2VkIGEgcmVncmVzc2lvbiBvciBhY3R1YWxseSBtYWRlIHRoZSB0ZXN0IGJl dHRlci4KKwogMjAyMS0wMi0yNSAgSmVyIE5vYmxlICA8amVyLm5vYmxlQGFwcGxlLmNvbT4KIAog ICAgICAgICBbQ29jb2FdIEFwcGVuZGluZyBhIHBhcnRpYWwgc2VnbWVudCBvZiBhIFdlYk0gYXVk aW8gZmlsZSByZXN1bHRzIGluIG5vIGFkZGl0aW9uYWwgc2FtcGxlcwpkaWZmIC0tZ2l0IGEvTGF5 b3V0VGVzdHMvcGxhdGZvcm0vbWFjL1Rlc3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9wbGF0 Zm9ybS9tYWMvVGVzdEV4cGVjdGF0aW9ucwppbmRleCBmMTUyMmVkYmQ5NmI4N2E1ZDU5Y2Q0NTFk OTdiMmQzMjUyYjJiN2NkLi4xNjg0NTE2NDUxMjM2MTQzNTBmZGNkMGNkZjE4MDY3Mjc0ZGRkMGQ0 IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvVGVzdEV4cGVjdGF0aW9ucwor KysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMvVGVzdEV4cGVjdGF0aW9ucwpAQCAtMTE1OCw4 ICsxMTU4LDYgQEAgd2Via2l0Lm9yZy9iLzE2MTE0NCBodHRwL3Rlc3RzL21lZGlhL2hscy9obHMt YWNjZXNzaWJsaXR5LWRlc2NyaWJlcy12aWRlby1tZW51LmgKIAogd2Via2l0Lm9yZy9iLzE2MTg4 MyBpbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL2RvbS9ub2Rlcy9Qcm9jZXNzaW5nSW5z dHJ1Y3Rpb24tZXNjYXBlcy0xLnhodG1sIFsgUGFzcyBGYWlsdXJlIF0KIAotd2Via2l0Lm9yZy9i LzE2MjMwOCBpbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL21lZGlhLXNvdXJjZS9tZWRp YXNvdXJjZS1kdXJhdGlvbi5odG1sIFsgUGFzcyBGYWlsdXJlIF0KLQogd2Via2l0Lm9yZy9iLzE2 MjUyMyBpbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL21lZGlhLXNvdXJjZS9tZWRpYXNv dXJjZS1wcmVsb2FkLmh0bWwgWyBQYXNzIEZhaWx1cmUgXQogCiB3ZWJraXQub3JnL2IvMTYyNjI2 IGltcG9ydGVkL3czYy93ZWItcGxhdGZvcm0tdGVzdHMvbWVkaWEtc291cmNlL21lZGlhc291cmNl LWxpdmVzZWVrYWJsZS5odG1sIFsgUGFzcyBGYWlsdXJlIF0KQEAgLTIyOTMsNCArMjI5MSw2IEBA IHdlYmtpdC5vcmcvYi8yMjIxOTcgbWVkaWEvbW9kZXJuLW1lZGlhLWNvbnRyb2xzL3RyYWNrcy1z dXBwb3J0L3RyYWNrcy1zdXBwb3J0LWF1CiAKIHdlYmtpdC5vcmcvYi8yMjIyMDUgY3NzMy9jYWxj L3RyYW5zZm9ybXMtdHJhbnNsYXRlLmh0bWwgWyBQYXNzIEltYWdlT25seUZhaWx1cmUgXQogCi13 ZWJraXQub3JnL2IvMjIyMTk1IGluc3BlY3Rvci9hbmltYXRpb24vbGlmZWN5Y2xlLWNzcy10cmFu c2l0aW9uLmh0bWwgWyBQYXNzIFRpbWVvdXQgXQpcIE5vIG5ld2xpbmUgYXQgZW5kIG9mIGZpbGUK K3dlYmtpdC5vcmcvYi8yMjIxOTUgaW5zcGVjdG9yL2FuaW1hdGlvbi9saWZlY3ljbGUtY3NzLXRy YW5zaXRpb24uaHRtbCBbIFBhc3MgVGltZW91dCBdCisKK3dlYmtpdC5vcmcvYi8yMjI0MjIgaW1w b3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy9tZWRpYS1zb3VyY2UvbWVkaWFzb3VyY2UtZHVy YXRpb24uaHRtbCBbIFBhc3MgRmFpbHVyZSBTbG93IF0KXCBObyBuZXdsaW5lIGF0IGVuZCBvZiBm aWxlCg==