Skip to content

Commit

Permalink
Force the usage of mp4 for media-element WPTs
Browse files Browse the repository at this point in the history
  • Loading branch information
ferjm committed Jan 9, 2019
1 parent 6c2c3f7 commit f7eeb26
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions components/script/dom/htmlmediaelement.rs
Expand Up @@ -314,7 +314,8 @@ impl HTMLMediaElement {
.task_manager()
.media_element_task_source()
.queue_simple_event(self.upcast(), atom!("timeupdate"), &window);
self.next_timeupdate_event.set(time::get_time() + Duration::milliseconds(350));
self.next_timeupdate_event
.set(time::get_time() + Duration::milliseconds(350));
}
}

Expand Down Expand Up @@ -1283,8 +1284,14 @@ impl HTMLMediaElementMethods for HTMLMediaElement {
// https://html.spec.whatwg.org/multipage/#dom-navigator-canplaytype
fn CanPlayType(&self, type_: DOMString) -> CanPlayTypeResult {
match type_.parse::<Mime>() {
// XXX GStreamer is currently not very reliable playing OGG and most of
// the media related WPTs uses OGG if we report that we are able to
// play this type. So we report that we are unable to play it to force
// the usage of other types.
// https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/520
Ok(ref mime)
if (mime.type_() == mime::APPLICATION && mime.subtype() == mime::OCTET_STREAM) =>
if (mime.type_() == mime::APPLICATION && mime.subtype() == mime::OCTET_STREAM) ||
(mime.subtype() == mime::OGG) =>
{
CanPlayTypeResult::_empty
},
Expand Down

0 comments on commit f7eeb26

Please sign in to comment.