If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Move MediaDecoder::mOutputStreams to DecodedStream.cpp/h

RESOLVED FIXED in Firefox 41

Status

()

Core
Audio/Video
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla41
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

3 years ago
Continue to move stream capture related code to its own place.
(Assignee)

Updated

3 years ago
Blocks: 1163467
(Assignee)

Updated

3 years ago
Depends on: 1163469
(Assignee)

Comment 1

3 years ago
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=eba61aae266a
(Assignee)

Updated

3 years ago
Blocks: 1163489
(Assignee)

Comment 2

2 years ago
Fix build errors.
Try again: https://treeherder.mozilla.org/#/jobs?repo=try&revision=21677437deb9
(Assignee)

Comment 3

2 years ago
Created attachment 8606865 [details] [diff] [review]
1163474_part1_add_DecodedStream-v1.patch

Part 1 - Create a wrapper class to which MediaDecoder::mOutputStreams will move.
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Attachment #8606865 - Flags: review?(roc)
(Assignee)

Comment 4

2 years ago
Created attachment 8606868 [details] [diff] [review]
1163474_part2_move_mOutputStreams-v1.patch

Part 2 - move MediaDecoder::mOutputStreams to DecodedStream.
Attachment #8606868 - Flags: review?(roc)
Attachment #8606865 - Flags: review?(roc) → review+
Attachment #8606868 - Flags: review?(roc) → review+
(Assignee)

Comment 5

2 years ago
Thanks for the review!

Comment 6

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/51a238090f5d
https://hg.mozilla.org/integration/mozilla-inbound/rev/843d5332761b
https://hg.mozilla.org/mozilla-central/rev/51a238090f5d
https://hg.mozilla.org/mozilla-central/rev/843d5332761b
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.