Closed Bug 1068996 Opened 8 years ago Closed 8 years ago

WMFMediaDataDecoder::Shutdown() doesn't shutdown

Categories

(Core :: Audio/Video, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla35

People

(Reporter: cpearce, Assigned: cpearce)

Details

Attachments

(1 file)

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.
Attached patch PatchSplinter Review
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.
Attachment #8491115 - Flags: review?(ajones)
Attachment #8491115 - Flags: review?(ajones) → review+
https://hg.mozilla.org/mozilla-central/rev/1674d241ce3d
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in before you can comment on or make changes to this bug.