MP4Demuxer::mResource is useless

RESOLVED INVALID

Status

()

RESOLVED INVALID
2 years ago
2 years ago

People

(Reporter: gerald, Assigned: gerald)

Tracking

53 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

2 years ago
The MP4Demuxer::mResource RefPtr is set once with a MediaResource and then never used.
That MediaResource is also stored in a RefPtr inside MP4Demuxer::mStream, therefore its lifetime will not be shortened.

So we can just get rid of MP4Demuxer::mResource.
(Assignee)

Comment 1

2 years ago
My mistake, it's actually used from MP4TrackDemuxer. I forgot to save a file before compiling, d'oh!
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.