Include all WebM codecs in transcode profile (squash me on submit)

This commit is contained in:
Chris Tam 2021-08-24 18:36:31 -04:00
parent f624326eb4
commit 78cf009713

View File

@ -667,38 +667,13 @@ import browser from './browser';
}
}
if (canPlayVp8) {
// TODO: Remove vpx entry once servers are migrated
if (webmAudioCodecs.length && webmVideoCodecs.length) {
profile.TranscodingProfiles.push({
Container: 'webm',
Type: 'Video',
AudioCodec: webmAudioCodecs.includes('opus') ? 'opus' : 'vorbis',
VideoCodec: 'vpx',
Context: 'Streaming',
Protocol: 'http',
// If audio transcoding is needed, limit channels to number of physical audio channels
// Trying to transcode to 5 channels when there are only 2 speakers generally does not sound good
MaxAudioChannels: physicalAudioChannels.toString()
});
profile.TranscodingProfiles.push({
Container: 'webm',
Type: 'Video',
AudioCodec: webmAudioCodecs.includes('opus') ? 'opus' : 'vorbis',
VideoCodec: 'vp8',
Context: 'Streaming',
Protocol: 'http',
// If audio transcoding is needed, limit channels to number of physical audio channels
// Trying to transcode to 5 channels when there are only 2 speakers generally does not sound good
MaxAudioChannels: physicalAudioChannels.toString()
});
}
if (canPlayVp9) {
profile.TranscodingProfiles.push({
Container: 'webm',
Type: 'Video',
AudioCodec: webmAudioCodecs.includes('opus') ? 'opus' : 'vorbis',
VideoCodec: 'vp9',
AudioCodec: webmAudioCodecs.join(','),
// TODO: Remove workaround when servers migrate away from 'vpx' for transcoding profiles.
VideoCodec: (canPlayVp8 ? webmVideoCodecs.concat('vpx') : webmVideoCodecs).join(','),
Context: 'Streaming',
Protocol: 'http',
// If audio transcoding is needed, limit channels to number of physical audio channels