Move MediaDecoder::FrameStatistics out of MediaDecoder

RESOLVED FIXED in Firefox 43

Status

()

Core
Audio/Video: Playback
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: kikuo, Assigned: kikuo)

Tracking

unspecified
mozilla43
All
Unspecified
Points:
---

Firefox Tracking Flags

(firefox43 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
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)

Updated

2 years ago
Assignee: nobody → kikuo
(Assignee)

Comment 1

2 years ago
Created attachment 8661277 [details]
MozReview Request: Bug 1204882, Move MediaDecoder::FrameStatistics out of MediaDecoder for easier use in other classes; r?jwwang

Bug 1204882, Move MediaDecoder::FrameStatistics out of MediaDecoder for easier use in other classes; r?jwwang
Attachment #8661277 - Flags: review?(jwwang)
(Assignee)

Updated

2 years ago
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
(Assignee)

Comment 3

2 years ago
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.
(Assignee)

Updated

2 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bb6aaac878a4
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
(Assignee)

Updated

2 years ago
No longer blocks: 1194918
You need to log in before you can comment on or make changes to this bug.