Closed
Bug 1242843
Opened 8 years ago
Closed 8 years ago
Make MDSM::mDecodedVideoEndTime zero-based
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: jwwang, Assigned: jwwang)
Details
Attachments
(1 file)
The same reason as bug 1242841.
Assignee | ||
Comment 1•8 years ago
|
||
try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f380d61359e6
Assignee: nobody → jwwang
Assignee | ||
Comment 2•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/32401/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/32401/
Attachment #8712001 -
Flags: review?(kikuo)
Comment 3•8 years ago
|
||
Comment on attachment 8712001 [details] MozReview Request: Bug 1242843 - Make MDSM::mDecodedVideoEndTime zero-based. r=kikuo. https://reviewboard.mozilla.org/r/32401/#review29297 ::: dom/media/MediaDecoderStateMachine.cpp:904 (Diff revision 1) > - mDecodedVideoEndTime = video ? video->GetEndTime() : mDecodedVideoEndTime; > + mDecodedVideoEndTime = video->GetEndTime(); It seems that no one even check the return value of |AdjustForStartTIme()|, is there any case in which the value from |video->GetEndTime()| becomes negative after |AdjustForStartTIme()|, i.e. badly muxed media file ?
Attachment #8712001 -
Flags: review?(kikuo)
Assignee | ||
Comment 4•8 years ago
|
||
https://reviewboard.mozilla.org/r/32401/#review29297 > It seems that no one even check the return value of |AdjustForStartTIme()|, is there any case in which the value from |video->GetEndTime()| becomes negative after |AdjustForStartTIme()|, i.e. badly muxed media file ? All video timestamps (start and end time) should be >= 0 after adjustment. However, we can't be sure of that for a badly muxed file. Therefore, we drop and won't render video frames with abnormal timestamps in VideoSink.
Comment 5•8 years ago
|
||
Comment on attachment 8712001 [details] MozReview Request: Bug 1242843 - Make MDSM::mDecodedVideoEndTime zero-based. r=kikuo. https://reviewboard.mozilla.org/r/32401/#review29311
Attachment #8712001 -
Flags: review+
Assignee | ||
Comment 6•8 years ago
|
||
Thanks!
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3ff4bda55715
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3ff4bda55715
You need to log in
before you can comment on or make changes to this bug.
Description
•