1068996 - WMFMediaDataDecoder::Shutdown() doesn't shutdown

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

nsresult
WMFMediaDataDecoder::Shutdown()
{
  mDecoder = nullptr;
  return NS_OK;
}

It just drops the reference to the MFT, but mMFTManager also has a reference, so the MFT won't actually die/shutdown.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch

Make the WMF MediaDataDecoder shutdown when it's told to... Previously we'd destroy it slightly later when the MP4Reader cleared its references to its MediaDataDecoders.

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1674d241ce3d

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1674d241ce3d