Closed
Bug 1330254
Opened 7 years ago
Closed 7 years ago
Move MDSM::Ensure{Audio,Video}DecodeTaskQueued 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
(4 files)
No description provided.
Assignee | ||
Updated•7 years ago
|
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8826020 -
Flags: review?(kikuo)
Attachment #8826021 -
Flags: review?(kikuo)
Attachment #8826022 -
Flags: review?(kikuo)
Attachment #8826023 -
Flags: review?(kikuo)
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8826020 [details] Bug 1330254. Part 1 - move MDSM::Ensure{Audio,Video}DecodeTaskQueued into DecodingState. https://reviewboard.mozilla.org/r/104064/#review104850
Attachment #8826020 -
Flags: review?(kikuo) → review+
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8826021 [details] Bug 1330254. Part 2 - move NeedToSkipToNextKeyframe() into DecodingState and remove some checks for mState. https://reviewboard.mozilla.org/r/104066/#review104878
Attachment #8826021 -
Flags: review?(kikuo) → review+
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8826023 [details] Bug 1330254. Part 4 - let Handle{Audio,Video}Canceled and Handle{Audio,Video}Waited call Request{Audio,Video}Data instead. https://reviewboard.mozilla.org/r/104070/#review104880
Attachment #8826023 -
Flags: review?(kikuo) → review+
Comment 8•7 years ago
|
||
mozreview-review |
Comment on attachment 8826022 [details] Bug 1330254. Part 3 - remove some checks from NeedToSkipToNextKeyframe(). https://reviewboard.mozilla.org/r/104068/#review104882 Looks good to me here.
Attachment #8826022 -
Flags: review?(kikuo) → review+
Assignee | ||
Comment 9•7 years ago
|
||
Thanks!
Comment 10•7 years ago
|
||
Pushed by jwwang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ab918aa8e59b Part 1 - move MDSM::Ensure{Audio,Video}DecodeTaskQueued into DecodingState. r=kikuo https://hg.mozilla.org/integration/autoland/rev/bfc46e0cfc33 Part 2 - move NeedToSkipToNextKeyframe() into DecodingState and remove some checks for mState. r=kikuo https://hg.mozilla.org/integration/autoland/rev/2d1240cfb011 Part 3 - remove some checks from NeedToSkipToNextKeyframe(). r=kikuo https://hg.mozilla.org/integration/autoland/rev/1670628877e1 Part 4 - let Handle{Audio,Video}Canceled and Handle{Audio,Video}Waited call Request{Audio,Video}Data instead. r=kikuo
Comment 11•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ab918aa8e59b https://hg.mozilla.org/mozilla-central/rev/bfc46e0cfc33 https://hg.mozilla.org/mozilla-central/rev/2d1240cfb011 https://hg.mozilla.org/mozilla-central/rev/1670628877e1
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
•