Bug 209353

Summary: (r258747) [ Mac wk1 Release ] media/video-background-tab-playback.html is failing.
Product: WebKit Reporter: Jason Lawrence <Lawrence.j>
Component: MediaAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: hector_i_lopez, tsavell, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Mac   
OS: macOS 10.14   
See Also: https://bugs.webkit.org/show_bug.cgi?id=196502
Attachments:
Description Flags
Patch none

Jason Lawrence
Reported 2020-03-20 12:04:06 PDT
media/video-background-tab-playback.html Description: This test is flaky failing on Mac wk1 Release. It appears to have regressed with the changes made to r258747. History: https://results.webkit.org/?suite=layout-tests&test=media%2Fvideo-background-tab-playback.html&platform=mac&flavor=wk1&style=release&limit=50000 Diff: --- /Volumes/Data/slave/catalina-release-tests-wk1/build/layout-test-results/media/video-background-tab-playback-expected.txt +++ /Volumes/Data/slave/catalina-release-tests-wk1/build/layout-test-results/media/video-background-tab-playback-actual.txt @@ -11,12 +11,6 @@ ** Simulate switching the tab to background, video should pause. RUN(testRunner.setPageVisibility('hidden')) -EXPECTED (video.paused == 'true') OK - -** Simulate switch back to foreground, video should remain paused. -RUN(internals.applicationWillEnterForeground()) - -EXPECTED (video.paused == 'true') OK - +Video did not pause after switching to backgound. FAIL END OF TEST
Attachments
Patch (2.14 KB, patch)
2020-08-03 08:43 PDT, Hector Lopez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2020-03-20 12:04:28 PDT
<rdar://problem/60697806>
Jason Lawrence
Comment 2 2020-03-20 12:14:39 PDT
I can reproduce this issue on r258747 with the command below. run-webkit-tests --iterations 1000 --exit-after-n-failures --force -f media/video-background-tab-playback.html r258747: [15/1000] media/video-background-tab-playback.html failed unexpectedly (text diff) [16/1000] media/video-background-tab-playback.html failed unexpectedly (text diff) [17/1000] media/video-background-tab-playback.html failed unexpectedly (text diff) Exiting early after 3 failures. 14 tests run. 11 tests ran as expected, 3 didn't (986 didn't run):
Jason Lawrence
Comment 3 2020-03-20 13:41:23 PDT
I have marked this test as failing while this issue is investigated further here: https://trac.webkit.org/changeset/258785/webkit
Hector Lopez
Comment 4 2020-08-03 08:31:51 PDT
media/video-background-tab-playback.html This test is also a flaky failure on macOS wk2 Release. First instance of failure on wk2 Release appears at r258793 according to history. History: https://results.webkit.org/?suite=layout-tests&test=media%2Fvideo-background-tab-playback.html&limit=50000&platform=mac&style=release Diff: --- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/media/video-background-tab-playback-expected.txt +++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/media/video-background-tab-playback-actual.txt @@ -11,12 +11,6 @@ ** Simulate switching the tab to background, video should pause. RUN(testRunner.setPageVisibility('hidden')) -EXPECTED (video.paused == 'true') OK - -** Simulate switch back to foreground, video should remain paused. -RUN(internals.applicationWillEnterForeground()) - -EXPECTED (video.paused == 'true') OK - +Video did not pause after switching to backgound. FAIL END OF TEST
Hector Lopez
Comment 5 2020-08-03 08:43:35 PDT
Created attachment 405832 [details] Patch
Truitt Savell
Comment 6 2020-08-03 09:03:16 PDT
Comment on attachment 405832 [details] Patch Clearing flags on attachment: 405832 Committed r265197: <https://trac.webkit.org/changeset/265197>
Note You need to log in before you can comment on or make changes to this bug.