1163489 - Remove dependency on MediaDecoder from DecodedStream

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[JW Wang [:jwwang] [:jw_wang]]

Refactoring continues...

Comment 1

[JW Wang [:jwwang] [:jw_wang]]

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3e0fabb953dd

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

Try again: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e3739ce93a7e

Comment 3

[JW Wang [:jwwang] [:jw_wang]]

Attachment: 1163489_remove_dependency_on_MediaDecoder-v1.patch

By passing MediaDecoder::mReentrantMonitor to the constructor of DecodedStream, we can now get rid of MediaDecoder from DecodedStream. This is a step forward to move DecodedStream related code out of MediaDecoder.

Comment 4

[JW Wang [:jwwang] [:jw_wang]]

Thanks for the review!

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e4d419067288

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/e4d419067288