Closed Bug 1204882 Opened 4 years ago Closed 4 years ago

Move MediaDecoder::FrameStatistics out of MediaDecoder

Categories

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

All
Unspecified
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: kikuo, Assigned: kikuo)

Details

Attachments

(1 file)

Based on Bug 1194918 Comment 20, move MediaDecoder::FrameStatistics out of MediaDecoder into its own file to make it easier for other classes usage. e.g. As a consumer, DecodedVideoDataSink should not know stuff of MediaDecoder but it would need a mFrameStatistics which is owned by MediaDecoder for updating frame statistics.
Assignee: nobody → kikuo
Bug 1204882, Move MediaDecoder::FrameStatistics out of MediaDecoder for easier use in other classes; r?jwwang
Attachment #8661277 - Flags: review?(jwwang)
Blocks: 1194918
Attachment #8661277 - Flags: review?(jwwang) → review+
Comment on attachment 8661277 [details]
MozReview Request: Bug 1204882, Move MediaDecoder::FrameStatistics out of MediaDecoder for easier use in other classes; r?jwwang

https://reviewboard.mozilla.org/r/19333/#review17377
try run seems ok, https://treeherder.mozilla.org/#/jobs?repo=try&revision=638867ebcef7

This is my first time using MozReivew, I'm not sure if there will be any problem that I've already made the request status from pending => submitted.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bb6aaac878a4
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
No longer blocks: 1194918
You need to log in before you can comment on or make changes to this bug.