MediaSource incorrectly reporting that mp4 is supported, even when it isn't.

RESOLVED FIXED in Firefox 42

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jya, Assigned: jya)

Tracking

(Blocks: 1 bug)

Trunk
mozilla42
Unspecified
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 affected, firefox42 fixed)

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

4 years ago
On Linux, we can only use MediaSource with mp4 if FFmpeg support has been compiled in, and FFmpeg external libraries are present.

However, MediaSource IsTypeSupported only checks if the type is mp4 or webm and if that the HTMLMediaElement can play such file.

With gstreamer installed, we would report that mp4 was supported, even though gstreamer isn't usable with MSE.

MediaSource should only report MP4 as supported if the MP4Reader is usable.
Jean-Yves, does this bug affect Windows XP or only Linux builds with FFmpeg compiled in?
status-firefox42: --- → affected
Flags: needinfo?(jyavenard)
OS: Unspecified → Linux
(Assignee)

Comment 2

4 years ago
AFAICT, Only Linux.

It affects plain linux build, you don't need ffmpeg built-in.
Flags: needinfo?(jyavenard)
(Assignee)

Comment 3

4 years ago
We now have identical results between Windows XP and Linux.
Attachment #8636407 - Flags: review?(karlt)
(Assignee)

Updated

4 years ago
Assignee: nobody → jyavenard
(Assignee)

Updated

4 years ago
Blocks: 1105760
(Assignee)

Updated

4 years ago
Depends on: 1171379
(Assignee)

Comment 6

4 years ago
We now have identical results between Windows XP and Linux.

Sorry for that, I had missed two tests.
Attachment #8636487 - Flags: review?(karlt)
Comment on attachment 8636438 [details] [diff] [review]
P1. Refactor DecoderTraits::CanHandleMediaType.

Review of attachment 8636438 [details] [diff] [review]:
-----------------------------------------------------------------

Suggestion: We could make this more OO by having a class that contains the codec list that we run queries on. Not sure if that would be better overall.

I'm fine with landing this patch as-is.

::: dom/media/DecoderTraits.cpp
@@ +191,5 @@
>  
>  /* static */ bool
>  DecoderTraits::IsWebMType(const nsACString& aType)
>  {
> +#ifdef MOZ_WEBM

Suggestion: We can probably get rid of the MOZ_WEBM configure flag.
Attachment #8636438 - Flags: review?(ajones) → review+
Comment on attachment 8636439 [details] [diff] [review]
P2. Properly reports if mimetype is support in mediasource.

Review of attachment 8636439 [details] [diff] [review]:
-----------------------------------------------------------------

I suspect creating a class that represents the parsed codec information would make the code cleaner. However it is still an improvement over what we've got. I'm OK with landing this patch as is.
Attachment #8636439 - Flags: review?(ajones) → review+

Updated

4 years ago
Blocks: 1184867
Attachment #8636407 - Attachment is obsolete: true
Attachment #8636407 - Flags: review?(karlt)
Attachment #8636487 - Flags: review?(karlt) → review+
https://hg.mozilla.org/mozilla-central/rev/75bedbf6633b
https://hg.mozilla.org/mozilla-central/rev/03b87d091470
https://hg.mozilla.org/mozilla-central/rev/340872ac89ce
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox42: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
This landing might be causing bug 1186965 which is a smoke test blocker so we might need this backed out.
Whiteboard: [backout-asap]
(Assignee)

Comment 12

4 years ago
Backing it out would be difficult, and I doubt this particular commit is what causes that assert. all that commit does is preventing MSE to report that MP4 is support on Linux.

More likely the new MSE.
(Assignee)

Updated

4 years ago
Whiteboard: [backout-asap]
(Assignee)

Updated

4 years ago
No longer depends on: 1186965
Depends on: 1188131
You need to log in before you can comment on or make changes to this bug.