Bug 209353 - (r258747) [ Mac wk1 Release ] media/video-background-tab-playback.html is failing.
Summary: (r258747) [ Mac wk1 Release ] media/video-background-tab-playback.html is fai...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: WebKit Nightly Build
Hardware: Mac macOS 10.14
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-03-20 12:04 PDT by Jason Lawrence
Modified: 2020-08-03 09:03 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.14 KB, patch)
2020-08-03 08:43 PDT, Hector Lopez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Lawrence 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
Comment 1 Radar WebKit Bug Importer 2020-03-20 12:04:28 PDT
<rdar://problem/60697806>
Comment 2 Jason Lawrence 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):
Comment 3 Jason Lawrence 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
Comment 4 Hector Lopez 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
Comment 5 Hector Lopez 2020-08-03 08:43:35 PDT
Created attachment 405832 [details]
Patch
Comment 6 Truitt Savell 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>