getMimeType() shoulnd’t expect a semicolon in the type attribute of every <source> element

[mathiasbynens]

The demo uses the following HTML:

<video poster="demo/poster.jpg" width="352" height="264" controls preload>
 <source src="demo/video.ogv" type='video/ogg; codecs="theora, vorbis"'></source>
 <source src="demo/video.mp4" type='video/mp4; codecs="avc1.42E01E, mp4a.40.2"'></source>
</video>

(By the way, you shouldn’t close the source element.) The codecs part is optional and can be omitted in HTML5:

<video poster="demo/poster.jpg" width="352" height="264" controls preload>
 <source src="demo/video.ogv" type="video/ogg">
 <source src="demo/video.mp4" type="video/mp4">
</video>

However, this causes html5media to break. The getMimeType() function relies on the semicolon (;) being there in the type attribute.

[etianen]

I've pushed a fix up to master, and will do a mini-release shortly. Oops.

In terms of closing the source element - I know the HTML5 specification doesn't like it, but some of the old browsers need it to be there else they completely fail. I did a load of testing when I started the project, and the syntax I use for the video tags, however clunky, is the only one that worked across the board. I wish I'd made some notes about what failed under what circumstances, but I didn't. In this case, I think it was Opera that needed the closing source tag, but don't quote me on that.