Remove OutputStreamListener from DecodedStream.cpp

RESOLVED FIXED in Firefox 43

Status

()

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

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla43
Points:
---
Dependency tree / graph

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
The class is introduced to fix bug 1128420. However, after bug 1201393 is landed, the output streams will no longer block the decoded stream and therefore we can remove the class now.
(Assignee)

Updated

2 years ago
Depends on: 1128420, 1201393
Priority: -- → P2
(Assignee)

Updated

2 years ago
Assignee: nobody → jwwang
(Assignee)

Comment 1

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

Comment 2

2 years ago
Created attachment 8659790 [details]
MozReview Request: Bug 1203449 - Remove OutputStreamListener from DecodedStream.cpp.

Bug 1203449 - Remove OutputStreamListener from DecodedStream.cpp.
Attachment #8659790 - Flags: review?(roc)
Comment on attachment 8659790 [details]
MozReview Request: Bug 1203449 - Remove OutputStreamListener from DecodedStream.cpp.

https://reviewboard.mozilla.org/r/19017/#review16887
Attachment #8659790 - Flags: review?(roc) → review+
(Assignee)

Comment 4

2 years ago
Thanks for the review!

Comment 5

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/4496829ff993
https://hg.mozilla.org/mozilla-central/rev/4496829ff993
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.