New issue
Advanced search Search tips

Issue 811605 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

media/video-poster-after-loadedmetadata.html failing layout tests.

Project Member Reported by khushals...@chromium.org, Feb 13 2018

Issue description

I have a WIP patch[1] that changes layout tests running in single-threaded mode to actually run through the update phases in the compositor during the main frame, which is closer to production behaviour. This is making media/video-poster-after-loadedmetadata.html fail consistently.

I ran the test with threaded compositing to verify if this was a bug with the patch itself or the test and I get the same failure when running with threaded compositing. The interesting part is that while threaded compositing makes it flaky, it fails quite consistently with the patch above. Seems like a timing issue which was racy in threaded mode but we hit consistently now.

mlamouri@, would you be the correct person to own this? I'm marking the test flaky for now.

[1]: https://chromium-review.googlesource.com/c/chromium/src/+/900205
 
Cc: mlamouri@chromium.org
Components: -Blink>Media>Video Blink>Media>Controls
Owner: steimel@chromium.org
Thanks for the notification Khushal. Assigning steimel@ to take a look.
Is this still an issue? I see that your CL has landed, I don't see the test listed on TestExpectations, and I'm unable to repro a failure
The CL got reverted and I'm going to try to reland once the CQ reopens: https://chromium-review.googlesource.com/c/chromium/src/+/918122. You can patch it locally to repro.
Labels: -Pri-1 Pri-2

Sign in to add a comment