Bug 167348 – LayoutTest media/modern-media-controls/media-controller/media-controller-auto-hide-mouse-enter-and-mouse-leave.html is a flaky failure

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Ryan Haddad]

LayoutTest media/modern-media-controls/media-controller/media-controller-auto-hide-mouse-enter-and-mouse-leave.html is a flaky failure

https://build.webkit.org/results/Apple%20Sierra%20Release%20WK2%20(Tests)/r211060%20(2908)/results.html

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=media%2Fmodern-media-controls%2Fmedia-controller%2Fmedia-controller-auto-hide-mouse-enter-and-mouse-leave.html

--- /Volumes/Data/slave/sierra-release-tests-wk2/build/layout-test-results/media/modern-media-controls/media-controller/media-controller-auto-hide-mouse-enter-and-mouse-leave-expected.txt
+++ /Volumes/Data/slave/sierra-release-tests-wk2/build/layout-test-results/media/modern-media-controls/media-controller/media-controller-auto-hide-mouse-enter-and-mouse-leave-actual.txt
@@ -7,7 +7,7 @@
 PASS controlsBar.classList.contains('faded') is false
 
 Auto-hide timer has elapsed, controls bar should be faded.
-PASS controlsBar.classList.contains('faded') is true
+FAIL controlsBar.classList.contains('faded') should be true. Was false.
 
 Mouse moved over the media, controls bar should be visible.
 PASS controlsBar.classList.contains('faded') is false

[Ryan Haddad]

Marked as flaky in https://trac.webkit.org/changeset/211073

[Antoine Quint]

This is tricky because this test, and a host of others, is timing-sensitive by design since it's testing the auto-hide timer. The test checks as close as possible to the auto-hide delay that the controls are still visible, and then again as soon as possible after the auto-hide delay has elapsed to check that the controls are no longer visible. So in certain cases, probably where the system is under pressure and timers run late, we get in cases where the check prior to the timer firing fails because we've actually run past auto-hide delay. We're going to have to find a different way to test this.

[Radar WebKit Bug Importer]

<rdar://problem/30822186>

[Antoine Quint]

Created attachment 303274 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 303274 [details]
Patch

Clearing flags on attachment: 303274

Committed r213340: <http://trac.webkit.org/changeset/213340>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.