2 PASS check MediaRecorder.canRecordMimeType() with video/invalid
3 FAIL check MediaRecorder.canRecordMimeType() with video/webm and vp8 assert_equa
ls: expected "maybe" but got ""
Peter Beverloo
2015/09/02 16:39:37
Out of interest, aren't these tests going to fail
Out of interest, aren't these tests going to fail when you implement the
Chromium-side of this change? If so, you may want to remove them now, and add
them after that lands. It saves you from a 3-sided patch, and means that you
don't have to check in failing expectations.
mcasas
2015/09/02 21:01:47
Yeah, this was a conundrum, I didn't want to land
On 2015/09/02 16:39:37, Peter Beverloo wrote:
> Out of interest, aren't these tests going to fail when you implement the
> Chromium-side of this change? If so, you may want to remove them now, and add
> them after that lands. It saves you from a 3-sided patch, and means that you
> don't have to check in failing expectations.
Yeah, this was a conundrum, I didn't want to land all this code
with no hints as to how it was supposed to work, in the form of
tests, but then I need to check the FAIL expectations. It'd be
great if I could commit a single CL that would "connect" with
the Chrome classes and update the ...-expected.txt, ideas?
4 PASS check MediaRecorder.canRecordMimeType() with audio/invalid
5 FAIL check MediaRecorder.canRecordMimeType() with audio/webm and opus assert_equ
als: expected "maybe" but got ""
Issue 1255873002: MediaRecorder Blink part
(Closed)
Created 5 years, 5 months ago by mcasas
Modified 5 years, 3 months ago
Reviewers: esprehn, Guido Urdaneta, mlamouri (slow - plz ping), Peter Beverloo
Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Comments: 208