virtual/android/fullscreen/video-controls-timeline.html flaky on linux |
||||
Issue descriptionhttps://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=video-controls-timeline.html steimel@ could you take a look and assign it to the right owner?
,
Apr 24 2018
At first glance, looks like the test is ending before the video updates from the seek. Maybe we should wait for a "seeked" event on the video instead of a "mouseup" on the timeline? Adding to list of P2s for broken layout tests.
,
May 1 2018
,
May 2 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a88eebcf319ecfb14cc2b8a788e6e1c7007609e2 commit a88eebcf319ecfb14cc2b8a788e6e1c7007609e2 Author: Tommy Steimel <steimel@chromium.org> Date: Wed May 02 00:35:52 2018 Fix flaky video-controls-timeline layout test This CL changes the fullscreen/video-controls-timeline.html layout test to wait for the "seeked" event on the video instead of just the "mouseup" event on the timeline to ensure that the video has updated correctly to match the expected image. Bug: 836241 Change-Id: Ia9a846db3eaba1452be3c21205f815da8f29893e Reviewed-on: https://chromium-review.googlesource.com/1038762 Reviewed-by: Becca Hughes <beccahughes@chromium.org> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#555251} [modify] https://crrev.com/a88eebcf319ecfb14cc2b8a788e6e1c7007609e2/third_party/WebKit/LayoutTests/TestExpectations [modify] https://crrev.com/a88eebcf319ecfb14cc2b8a788e6e1c7007609e2/third_party/WebKit/LayoutTests/fullscreen/video-controls-timeline-expected.txt [modify] https://crrev.com/a88eebcf319ecfb14cc2b8a788e6e1c7007609e2/third_party/WebKit/LayoutTests/fullscreen/video-controls-timeline.html
,
May 2 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by bugdroid1@chromium.org
, Apr 24 2018