227367 2021-06-24 11:19:58 -0700 REGRESSION (r278746): [Mac wk1] media/media-fullscreen-return-to-inline.html is a flaky timeout 2021-06-28 12:02:59 -0700 1 1 1 Unclassified WebKit Media WebKit Nightly Build Unspecified Unspecified REOPENED https://bugs.webkit.org/show_bug.cgi?id=226899 https://bugs.webkit.org/show_bug.cgi?id=226701 https://bugs.webkit.org/show_bug.cgi?id=226734 https://bugs.webkit.org/show_bug.cgi?id=226672 https://bugs.webkit.org/show_bug.cgi?id=157912 InRadar P2 Normal --- 1 ehutchison cdumez cdumez ehutchison peng.liu6 tsavell webkit-bot-watchers-bugzilla webkit-bug-importer wenson_hsieh oldest_to_newest 1772377 0 ehutchison 2021-06-24 11:19:58 -0700 media/media-fullscreen-return-to-inline.html flaky timeout on Mac wk1 queues. It appears timeout began around r278753. History: https://results.webkit.org/?suite=layout-tests&suite=layout-tests&test=media%2Fmedia-fullscreen-return-to-inline.html&test=media%2Fmedia-fullscreen-return-to-inline.html 1772379 1 webkit-bug-importer 2021-06-24 11:20:32 -0700 <rdar://problem/79733822> 1772751 2 ehutchison 2021-06-25 13:43:07 -0700 Was able to bisect issue to available builds 278743 to 278751. 1772753 3 tsavell 2021-06-25 13:45:06 -0700 Looking at the revisions in this range, it seems like either r278746 or r278747 could have caused this. Chris or Wenson could you see if one of your changes could be responsible for this regression? 1772754 4 wenson_hsieh 2021-06-25 13:47:19 -0700 (In reply to Truitt Savell from comment #3) > Looking at the revisions in this range, it seems like either r278746 or > r278747 could have caused this. > > Chris or Wenson could you see if one of your changes could be responsible > for this regression? It's definitely not r278747, since the codepaths that were changed there aren't compiled on macOS earlier than Monterey. 1772776 5 ap 2021-06-25 15:30:52 -0700 Re-titled based on above comments. 1772797 6 432305 ehutchison 2021-06-25 16:06:32 -0700 Created attachment 432305 Expectations 1772829 7 ews-feeder 2021-06-25 17:24:05 -0700 Committed r279302 (239178@main): <https://commits.webkit.org/239178@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 432305. 1772837 8 ryanhaddad 2021-06-25 17:46:52 -0700 Reopening since the patch only updated test expectations. 1773083 9 cdumez 2021-06-28 08:50:33 -0700 --- /Volumes/Data/worker/catalina-release-tests-wk1/build/layout-test-results/media/media-fullscreen-return-to-inline-expected.txt +++ /Volumes/Data/worker/catalina-release-tests-wk1/build/layout-test-results/media/media-fullscreen-return-to-inline-actual.txt @@ -1,3 +1,4 @@ +FAIL: Timed out waiting for notifyDone to be called Test that a <video> pauses after exiting fullscreen when the internal property "allowsInlineMediaPlaybackAfterFullscreen" is false. @@ -11,15 +12,6 @@ EVENT(webkitpresentationmodechanged) EXPECTED (internals.isChangingPresentationMode(video) == 'false') OK EXPECTED (video.webkitDisplayingFullscreen == 'false') OK -EXPECTED (video.paused == 'true') OK +EXPECTED (video.paused == 'true'), OBSERVED 'false' FAIL RUN(video.play()) -EVENT(webkitpresentationmodechanged) -EXPECTED (internals.isChangingPresentationMode(video) == 'false') OK -EXPECTED (video.webkitDisplayingFullscreen == 'true') OK -RUN(video.webkitExitFullscreen()) -EVENT(webkitpresentationmodechanged) -EXPECTED (internals.isChangingPresentationMode(video) == 'false') OK -EXPECTED (video.webkitDisplayingFullscreen == 'false') OK -EXPECTED (video.paused == 'false') OK -END OF TEST 1773103 10 cdumez 2021-06-28 09:51:22 -0700 Sadly, I am having trouble reproducing. I tried both release and debug builds with WK1. I tried running tests sequentially or in parallel. 432305 2021-06-25 16:06:32 -0700 2021-06-25 17:46:59 -0700 Expectations bug-227367-20210625160631.patch text/plain 1617 ehutchison U3VidmVyc2lvbiBSZXZpc2lvbjogMjc5MTc0CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggMzgwODQ2OWU3MTViYzcyYzFkMjU2 ZGVmYTJmNGNjNzgxMjllNzQ0YS4uZjYwNTIxYzEwOGJjZDdlZjlkYWYyN2JjY2Y2YjgyMTg4NTg5 M2M1MiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEyIEBACisyMDIxLTA2LTI1ICBFcmljIEh1dGNoaXNvbiAg PGVodXRjaGlzb25AYXBwbGUuY29tPgorCisgICAgICAgIFJFR1JFU1NJT04gKHIyNzg3NDYpOiBb TWFjIHdrIDFdIG1lZGlhL21lZGlhLWZ1bGxzY3JlZW4tcmV0dXJuLXRvLWlubGluZS5odG1sIGlz IGEgZmxha3kgdGltZW91dC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTIyNzM2Ny4KKworICAgICAgICBVbnJldmlld2VkIHRlc3QgZ2FyZGVuaW5nLgor CisgICAgICAgICogcGxhdGZvcm0vbWFjLXdrMS9UZXN0RXhwZWN0YXRpb25zOgorCiAyMDIxLTA2 LTIzICBBbWlyIE1hcmsgSnIgIDxhbWlyX21hcmtAYXBwbGUuY29tPgogCiAgICAgICAgIFttYWNP UyBXSzFdIGltcG9ydGVkL3czYy93ZWItcGxhdGZvcm0tdGVzdHMvY3NzL2Nzcy1zY3JvbGwtc25h cC9zY3JvbGwtdGFyZ2V0LXNuYXAtMDAzLmh0bWwgaXMgYSBmbGFreSBmYWlsdXJlLgpkaWZmIC0t Z2l0IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjLXdrMS9UZXN0RXhwZWN0YXRpb25zIGIvTGF5 b3V0VGVzdHMvcGxhdGZvcm0vbWFjLXdrMS9UZXN0RXhwZWN0YXRpb25zCmluZGV4IDQzNWNkYjAy Y2MzZWNmMmIwN2IxNGMwMWYyMWI0OTlhNzkwYzQyMjIuLmE4YTAzMTdhNzZiZTkxMTIxM2M1NTM1 YTI3ZDc3YTA1MTg5MDRjZGUgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL3BsYXRmb3JtL21hYy13 azEvVGVzdEV4cGVjdGF0aW9ucworKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtd2sxL1Rl c3RFeHBlY3RhdGlvbnMKQEAgLTEyOTUsNiArMTI5NSw4IEBAIHdlYmtpdC5vcmcvYi8yMjY1MTcg WyBCaWdTdXIgXSBtZWRpYS9tZWRpYS1ldmVudC1saXN0ZW5lcnMuaHRtbCBbIFBhc3MgVGltZW91 dCBdCiBbIGFybTY0IF0gaHR0cC90ZXN0cy9zZWN1cml0eS9zdG9yYWdlLWJsb2NraW5nLWxvb3Nl bmVkLXByaXZhdGUtYnJvd3NpbmctcGx1Z2luLmh0bWwgWyBTa2lwIF0KIFsgYXJtNjQgXSBodHRw L3Rlc3RzL3NlY3VyaXR5L3N0b3JhZ2UtYmxvY2tpbmctc3RyZW5ndGhlbmVkLXByaXZhdGUtYnJv d3NpbmctcGx1Z2luLmh0bWwgWyBTa2lwIF0KIAord2Via2l0Lm9yZy9iLzIyNzM2NyBtZWRpYS9t ZWRpYS1mdWxsc2NyZWVuLXJldHVybi10by1pbmxpbmUuaHRtbCBbIFBhc3MgVGltZW91dCBdCisK IHdlYmtpdC5vcmcvYi8yMjcwODUgaW1wb3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy9jc3Mv Y3NzLXNjcm9sbC1zbmFwL3Njcm9sbC10YXJnZXQtc25hcC0wMDMuaHRtbCBbIFBhc3MgSW1hZ2VP bmx5RmFpbHVyZSBdCiAKICMgd2Via2l0Lm9yZy9iLzIyNzIzNyBVcGRhdGluZyB0ZXN0IGV4cGVj dGF0aW9ucyBmb3IgMyB0ZXN0cy4K