1041861 - Clean up the H.264/AAC whitelisting code in HTML media

Status: RESOLVED WORKSFORME

(Core :: Audio/Video: Playback, defect)

Description

[cajbir (:cajbir)]

The detection and whitelisting of H.264/AAC codecs is spread across numerous places in the media code. There is, and not limited too, the following places where codecs are repeated:

content/media/gstreamer/GStreamerFormatHelper.cpp
content/media/DecoderTraits.cpp
content/media/fmp4/MP4Decoder.cpp
media/omx-plugin/OmxPlugin.cpp

Much of this code repeats strings and needs to be updated in multiple places. 

There is also code in content/media/mediasource/MediaSource.cpp that defines the codecs allowed in MSE (in gMediaSourceTypes).

This code should be refactored to make sure that there is a single place to be edited for adding supported codecs and formats and for whitelisting or preventing codecs from being used. 

Backends that support multiple formats should use the platform facilities to confirm existence of support for the requested format.

Comment 1

[Matthew Gregan [:kinetik]]

Also see http://src.chromium.org/viewvc/chrome?revision=277386&view=revision for changes made to Blink recently that has caused some fallout.

Comment 2

[Jean-Yves Avenard [:jya]]

this was done with the MediaMimeType object