Open Bug 1399023 Opened 8 years ago Updated 3 years ago

Check if media type contains 'codecs' in each decoder's 'IsSupportedType' call from 'CanHandleCodecsType()'.

Categories

(Core :: Audio/Video: Playback, enhancement, P3)

Product:
Core
Component:
Audio/Video: Playback
Type:
enhancement

Tracking

()

People

(Reporter: JamesCheng, Unassigned)

References

Details

Spawn by bug 1398102 comment 11 Ideally, we should handle missing `codecs` in each decoder's `IsSupportedType` call from `CanHandleCodecsType()`.
Quick thoughts while it's in my mind: I'm thinking IsSupportedType should probably return CanPlayStatus instead of just bool, in order to properly deal with the given (or not) codecs parameter. In which case the CanHandleMediaType/CanHandleCodecsType pair of functions in DecoderTraits.cpp could probably be simplified. For completeness we should also check that we correctly handle strange cases, e.g., `codecs=","` -- see bug 1398102 comment 24.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.