Closed
Bug 1328547
Opened 7 years ago
Closed 7 years ago
Move MDSM::Dispatch{Audio,Video}DecodeTaskIfNeeded into DecodingState
Categories
(Core :: Audio/Video: Playback, defect, P3)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: jwwang, Assigned: jwwang)
References
Details
Attachments
(3 files)
For DecodingState is the only caller.
Assignee | ||
Updated•7 years ago
|
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8823913 [details] Bug 1328547. part 1 - Move MDSM::Dispatch{Audio,Video}DecodeTaskIfNeeded into DecodingState. https://reviewboard.mozilla.org/r/102376/#review102818
Attachment #8823913 -
Flags: review?(kaku) → review+
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8823914 [details] Bug 1328547. part 2 - remove checks for IsShutdown() which is false in DecodingState. https://reviewboard.mozilla.org/r/102378/#review102820
Attachment #8823914 -
Flags: review?(kaku) → review+
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8823915 [details] Bug 1328547. part 3 - inline NeedToDecode{Audio,Video} and remove some checks. https://reviewboard.mozilla.org/r/102380/#review102822
Attachment #8823915 -
Flags: review?(kaku) → review+
Assignee | ||
Comment 7•7 years ago
|
||
Thanks!
Pushed by jwwang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0c8699ee3348 part 1 - Move MDSM::Dispatch{Audio,Video}DecodeTaskIfNeeded into DecodingState. r=kaku https://hg.mozilla.org/integration/autoland/rev/90ec1757f7c1 part 2 - remove checks for IsShutdown() which is false in DecodingState. r=kaku https://hg.mozilla.org/integration/autoland/rev/ebe4694e8c33 part 3 - inline NeedToDecode{Audio,Video} and remove some checks. r=kaku
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0c8699ee3348 https://hg.mozilla.org/mozilla-central/rev/90ec1757f7c1 https://hg.mozilla.org/mozilla-central/rev/ebe4694e8c33
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in
before you can comment on or make changes to this bug.
Description
•