WMFMediaDataDecoder::Shutdown() doesn't shutdown

RESOLVED FIXED in mozilla35

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cpearce, Assigned: cpearce)

Tracking

Trunk
mozilla35
x86_64
Windows 8.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
Posted 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
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in before you can comment on or make changes to this bug.