Bug 1350148: [AAC] Don't set channel count on output decoder. r?jya draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Fri, 24 Mar 2017 09:48:07 +0100
changeset 504478 a491242d24cefc63ddca99c243ceab1c400c7220
parent 503825 439ee2e4fc4ed4a82102608639d9751e376d5536
child 550654 aa5fadbf6cc49b0577644918f6451176d60dd258
push id50801
push userbmo:jyavenard@mozilla.com
push dateFri, 24 Mar 2017 08:53:17 +0000
reviewersjya
bugs1350148, 1347101
milestone55.0a1
Bug 1350148: [AAC] Don't set channel count on output decoder. r?jya Windows AAC decoder decode a mono AAC stream into a stereo PCM. Bug 1347101 set the output to be mono, which caused a failure to find the appropriate IMFMediaType. This partially revert bug 1347101 audio changes. MozReview-Commit-ID: 2M4X4rKKvXl
dom/media/platforms/wmf/WMFAudioMFTManager.cpp
--- a/dom/media/platforms/wmf/WMFAudioMFTManager.cpp
+++ b/dom/media/platforms/wmf/WMFAudioMFTManager.cpp
@@ -175,22 +175,16 @@ WMFAudioMFTManager::Init()
   NS_ENSURE_TRUE(SUCCEEDED(hr), false);
 
   hr = outputType->SetGUID(MF_MT_SUBTYPE, MFAudioFormat_Float);
   NS_ENSURE_TRUE(SUCCEEDED(hr), false);
 
   hr = outputType->SetUINT32(MF_MT_AUDIO_BITS_PER_SAMPLE, 32);
   NS_ENSURE_TRUE(SUCCEEDED(hr), false);
 
-  hr = outputType->SetUINT32(MF_MT_AUDIO_NUM_CHANNELS, mAudioChannels);
-  NS_ENSURE_TRUE(SUCCEEDED(hr), false);
-
-  hr = outputType->SetUINT32(MF_MT_ALL_SAMPLES_INDEPENDENT, 1);
-  NS_ENSURE_TRUE(SUCCEEDED(hr), false);
-
   hr = decoder->SetMediaTypes(inputType, outputType);
   NS_ENSURE_TRUE(SUCCEEDED(hr), false);
 
   mDecoder = decoder;
 
   return true;
 }