Bug 173119 - Layout Test media/navigate-with-pip-should-not-crash.html is a flaky failure.
Summary: Layout Test media/navigate-with-pip-should-not-crash.html is a flaky failure.
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-06-08 15:44 PDT by Matt Lewis
Modified: 2018-05-04 11:11 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Lewis 2017-06-08 15:44:57 PDT
media/navigate-with-pip-should-not-crash.html will intermittently fail.

Currently the dashboard states it is only failing on Sierra WK2 Release.
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=media%2Fnavigate-with-pip-should-not-crash.html

The test first failed on 5/25/2017 with this build: https://build.webkit.org/builders/Apple%20Sierra%20Release%20WK2%20(Tests)/builds/1776

Current build failure:
https://build.webkit.org/results/Apple%20Sierra%20Release%20WK2%20(Tests)/r217945%20(2084)/results.html
https://build.webkit.org/builders/Apple%20Sierra%20Release%20WK2%20(Tests)/builds/2084

Diff:
--- /Volumes/Data/slave/sierra-release-tests-wk2/build/layout-test-results/media/navigate-with-pip-should-not-crash-expected.txt
+++ /Volumes/Data/slave/sierra-release-tests-wk2/build/layout-test-results/media/navigate-with-pip-should-not-crash-actual.txt
@@ -5,6 +5,8 @@
 Going into Picture-in-Picture
 EXPECTED (mediaElement.webkitPresentationMode == 'picture-in-picture') OK
 EXPECTED (mediaElement.paused == 'false') OK
+EXPECTED (mediaElement.webkitPresentationMode == 'picture-in-picture'), OBSERVED 'inline' FAIL
+EXPECTED (mediaElement.paused == 'false') OK
 EXPECTED (videoframe.contentDocument.location == 'about:blank') OK
 END OF TEST
Comment 1 Alexey Proskuryakov 2017-06-13 13:14:18 PDT
 EXPECTED (mediaElement.paused == 'false') OK
+EXPECTED (mediaElement.webkitPresentationMode == 'picture-in-picture'), OBSERVED 'inline' FAIL
+EXPECTED (mediaElement.paused == 'false') OK


Does this mean that we are getting extra callbacks? The only change is that we get extra checks that fail, no existing ones fail.
Comment 2 Ryan Haddad 2018-05-03 12:49:25 PDT
This test has become far more flaky recently.

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=media%2Fnavigate-with-pip-should-not-crash.html

--- /Volumes/Data/slave/highsierra-debug-tests-wk2/build/layout-test-results/media/navigate-with-pip-should-not-crash-expected.txt
+++ /Volumes/Data/slave/highsierra-debug-tests-wk2/build/layout-test-results/media/navigate-with-pip-should-not-crash-actual.txt
@@ -5,6 +5,10 @@
 Going into Picture-in-Picture
 EXPECTED (mediaElement.webkitPresentationMode == 'picture-in-picture') OK
 EXPECTED (mediaElement.paused == 'false') OK
+EXPECTED (mediaElement.webkitPresentationMode == 'picture-in-picture'), OBSERVED 'inline' FAIL
+EXPECTED (mediaElement.paused == 'false') OK
+EXPECTED (mediaElement.webkitPresentationMode == 'picture-in-picture'), OBSERVED 'inline' FAIL
+EXPECTED (mediaElement.paused == 'false') OK
 EXPECTED (videoframe.contentDocument.location == 'about:blank') OK
 END OF TEST
Comment 3 Ryan Haddad 2018-05-04 11:11:00 PDT
Marked test as flaky in https://trac.webkit.org/r231363