Adding a few new MSE tests based on the spec.

Adding a few new MSE tests based on the spec.

This is intended to make the MSE implementation more spec compliant.
More such CLs to follow after this initial round is checked in.

BUG=
TEST=new test pass

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=173667

[prabhur1, 2014-05-07 18:27:25]

PTAL.
Adding a few MSE tests for spec compliancy.
Looking for early feedback before adding more tests.

Thanks!
Prabhu

[acolwell GONE FROM CHROMIUM, 2014-05-08 15:16:31]

lgtm % a few nits

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
File LayoutTests/http/tests/media/media-source/mediasource-addsourcebuffer.html
(right):

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-addsourcebuffer.html:78:
var mimetype = 'video/webm;codecs="vp8"';
Please use MediaSourceUtil.VIDEO_ONLY_TYPE so this test is codec independent.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-addsourcebuffer.html:91:
var mimetype = 'audio/webm;codecs="vorbis"';
Please use MediaSourceUtil.AUDIO_ONLY_TYPE so this test is codec independent.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
File
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html
(right):

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html:49:
"endofStream() threw an exception when in ended state");
nit: Indent is off. Should be 4 spaces like above.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html:63:
function() {  mediaSource.endOfStream("decode"); },
ditto

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html:78:
function() {  mediaSource.endOfStream("network"); },
ditto

[prabhur1, 2014-05-08 17:59:21]

Thanks for taking a look! Addressed the minor changes.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
File LayoutTests/http/tests/media/media-source/mediasource-addsourcebuffer.html
(right):

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-addsourcebuffer.html:78:
var mimetype = 'video/webm;codecs="vp8"';
On 2014/05/08 15:16:32, acolwell wrote:
> Please use MediaSourceUtil.VIDEO_ONLY_TYPE so this test is codec independent.

Done.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-addsourcebuffer.html:91:
var mimetype = 'audio/webm;codecs="vorbis"';
On 2014/05/08 15:16:32, acolwell wrote:
> Please use MediaSourceUtil.AUDIO_ONLY_TYPE so this test is codec independent.

Done.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
File
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html
(right):

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html:49:
"endofStream() threw an exception when in ended state");
On 2014/05/08 15:16:32, acolwell wrote:
> nit: Indent is off. Should be 4 spaces like above.

Done.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html:63:
function() {  mediaSource.endOfStream("decode"); },
On 2014/05/08 15:16:32, acolwell wrote:
> ditto

Done.

https://codereview.chromium.org/267333006/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-endofstream-invaliderror.html:78:
function() {  mediaSource.endOfStream("network"); },
On 2014/05/08 15:16:32, acolwell wrote:
> ditto

Done.

[prabhur1, 2014-05-08 18:00:53]

The CQ bit was checked by prabhur@chromium.org

[commit-bot: I haz the power, 2014-05-08 18:02:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/prabhur@chromium.org/267333006/20001

[commit-bot: I haz the power, 2014-05-08 19:19:31]

Change committed as 173667