mSourceLoadCandidate points to the child of the media element, and once the mParent is made strong, there is a cycle which cycle collector doesn't know about.
The patch has the assumption that if mSourceLoadCandidate is set null before NotifyLoadError or DecodeError, we have actually unlinked, and should not call QueueLoadFromSourceTask, since that takes a strong ref to the ownerDocument (which would cause leaks again).
Attachment #528840 - Flags: review?(chris)
Attachment #528840 - Flags: review?(chris) → review+
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.