Re: [media-source-tests] Added idlharness tests for the MSE spec. (#377) from Philippe Le Hegaret on 2013-12-12 (public-web-platform-tests-notifications@w3.org from December 2013)

From: Philippe Le Hegaret <web-platform-tests-notifications@w3.org>
Date: Thu, 12 Dec 2013 14:00:17 GMT
To: public-web-platform-tests-notifications@w3.org
Message-Id: <<web-platform-tests_issue_377_1386856817850@w3.org>>

Two initial comments:

Instead of /resources/webidl2/lib/webidl2.js, use /resources/WebIDLParser.js

sourceBuffer = mediaSource.addSourceBuffer("video/webm"); can throw an error. You need to make codec agnostic by using /common/media.js and getVideoURI, and catch the error so that you fail gracely. (media.js doesn't seem to handle webm yet?!?)


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

Received on Thursday, 12 December 2013 14:00:31 UTC