Re: [media-source-tests] Tests for Section 2. MediaSource Object API in Media Source Extensions. (#823)

Thank you, @shishimaru for producing this PR so long ago. In the interim, many other sources, especially Chromium, W3C, Edge and Firefox folks have updated the test suite such that much of what is covered in tests by this PR is currently already covered in other tests, unified with some helper utilities to simplify maintainability too (mediasource-utils.js).

Examining this PR vs current test suite, it looks like only the following pieces are missing from the test suite (but in this PR):
1. addSourceBuffer(null) should result in TypeError (not NotSupportedError as currently expected by mediasource-addsourcebuffer.html)
2. isTypeSupported(mime) returning true should mean that canPlayType(mime) is either "maybe" or "probably" (and certainly not ""). This can be added to mediasource-is-type-supported.html
3. Explicitly test isTypeSupported for each of: "xxx", "text/html", "image/jpeg" result in false. These can be added to mediasource-is-type-supported.html.
4. Explicitly test sourceBuffers added to distinct MediaSource objects give error when they are used in removeSourceBuffer to the wrong MediaSource. This can be added to mediasource-removesourcebuffer.html

I will take care of fixing these 4 gaps in a new PR. Thank you, again @shishimaru, for putting these tests together so long ago. Please reactivate this PR if you have any related concern to my decision to close it and fix the known gaps in a smaller separate PR.

-edit: I've prepared https://github.com/w3c/web-platform-tests/pull/3635 to fix these 4 gaps.

View on GitHub: https://github.com/w3c/web-platform-tests/pull/823#issuecomment-243626531

Received on Friday, 2 September 2016 21:08:06 UTC