Closed
Bug 1328782
Opened 8 years ago
Closed 8 years ago
Somewhat simplify the calculation of MediaDecoderStateMachine::HaveEnoughDecodedVideo()
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)
No description provided.
|
Assignee | |
Updated•8 years ago
|
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Updated•8 years ago
|
Summary: Somewhat simplify the calculation of MediaDecoderStateMachine::HaveEnoughDecodedAudio() → Somewhat simplify the calculation of MediaDecoderStateMachine::HaveEnoughDecodedVideo()
|
||
Comment 4•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8823933 [details]
Bug 1328782. part 1 - some tweaks to the expression.
https://reviewboard.mozilla.org/r/102398/#review103378
Attachment #8823933 -
Flags: review?(kikuo) → review+
|
|
||
Comment 5•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8823934 [details]
Bug 1328782. part 2 - some more tweaks to the expression.
https://reviewboard.mozilla.org/r/102400/#review103380
Attachment #8823934 -
Flags: review?(kikuo) → review+
|
|
||
Comment 6•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8823935 [details]
Bug 1328782. part 3 - remove the |VideoQueue().GetSize() > 0| check.
https://reviewboard.mozilla.org/r/102402/#review103386
Attachment #8823935 -
Flags: review?(kikuo) → review+
|
|
Assignee | |
Comment 7•8 years ago
|
||
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/755b68896241
part 1 - some tweaks to the expression. r=kikuo
https://hg.mozilla.org/integration/autoland/rev/a935de2c2a44
part 2 - some more tweaks to the expression. r=kikuo
https://hg.mozilla.org/integration/autoland/rev/c7f5fa45d3ae
part 3 - remove the |VideoQueue().GetSize() > 0| check. r=kikuo
|
||
Comment 9•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/755b68896241
https://hg.mozilla.org/mozilla-central/rev/a935de2c2a44
https://hg.mozilla.org/mozilla-central/rev/c7f5fa45d3ae
Status: NEW → RESOLVED
Closed: 8 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
•