Remove usage of decoder monitor from MDSM

RESOLVED FIXED in Firefox 44

Status

()

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

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla44
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox44 fixed)

Details

MozReview Requests

()

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

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Since all MDSM methods run on the task queue thread, we can now remove the usage of the decoder monitor.
(Assignee)

Updated

2 years ago
Assignee: nobody → jwwang
Blocks: 1146482
(Assignee)

Comment 1

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

Updated

2 years ago
Depends on: 1209864
(Assignee)

Comment 2

2 years ago
Created attachment 8669527 [details]
MozReview Request: Bug 1208934 - Remove usage of decoder monitor from MDSM. r=kinetik.

Bug 1208934 - Remove usage of decoder monitor from MDSM. r=kinetik.
Attachment #8669527 - Flags: review?(kinetik)
Comment on attachment 8669527 [details]
MozReview Request: Bug 1208934 - Remove usage of decoder monitor from MDSM. r=kinetik.

https://reviewboard.mozilla.org/r/21223/#review19195
Attachment #8669527 - Flags: review?(kinetik) → review+
Priority: -- → P2
(Assignee)

Comment 4

2 years ago
Thanks for the review!

Comment 5

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