Closed Bug 1405531 Opened 3 years ago Closed 3 years ago

Remove unused MediaDataDemuxer::HasTrackType

Categories

(Core :: Audio/Video: Playback, enhancement)

58 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: gerald, Assigned: gerald)

Details

Attachments

(1 file)

The virtual function MediaDataDemuxer::HasTrackType is never called, and the only used override in MediaSourceDemuxer can trivially be folded into its caller GetNumberTracks in the same class.

I'll remove it, so we don't have to worry about maintaining it anymore.
Comment on attachment 8914994 [details]
Bug 1405531 - Remove unused MediaDataDemuxer::HasTrackType -

https://reviewboard.mozilla.org/r/186258/#review191420
Attachment #8914994 - Flags: review?(jwwang) → review+
Pushed by gsquelart@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9edb781a9058
Remove unused MediaDataDemuxer::HasTrackType - r=jwwang
https://hg.mozilla.org/mozilla-central/rev/9edb781a9058
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.