Remove TestMediaStreamVideoRenderer usage from LayoutTests |
|||
Issue descriptionWhen running layout tests, there isn't a real video renderer for media streams but a TestMediaStreamVideoRenderer implementation instead. This implementation generates a stream of white frames and sends them in certain intervals. Video content aren't the real content and video events are also useless. We came across to this issue again on https://codereview.chromium.org/2768683002/#msg16. Do you know why this is implemented? AFAICT, plugging WebMediaPlayerMS instead works in Linux and outputs real video content.
,
Apr 5 2017
emircan@: I don't know anything about it. Perhaps perkj@ knows something about it?
,
Apr 26 2017
emircan - I'm assigning back to you since it's not clear who's the right person. I dug around and found this CL: https://codereview.chromium.org/2005053006 Which is on the blamelist for 'class TestMediaStreamVideoRenderer' and was reviewed by you (it's a naming cleanup CL so I figured the reviewer would be a more appropriate owner than the author).
,
Apr 26 2017
tommi@ sure I can take the bug. I am still not sure about why this class is there though. Currently, webkit tests aren't covering most of the code because of this. It goes back to below CL where both people left, so I am trying to find if anybody else knows about it. https://chromiumcodereview.appspot.com/10918052 |
|||
►
Sign in to add a comment |
|||
Comment 1 by emir...@chromium.org
, Mar 30 2017Status: Unconfirmed (was: Available)