https://hg.mozilla.org/mozilla-central/diff/116406daf772/content/media/MediaDecoderStateMachine.cpp The change to MDSM is a workaround to avoid genlock_lock_buffer error on Unagi. https://bugzilla.mozilla.org/show_bug.cgi?id=951278#c16 The genlock_lock_buffer error happened on Unagi only. I think we can now remove the workaround since MediaDecoderStateMachine::RenderVideoFrame() checks to ensure duplicated and invalid frames are not rendered.
Try looks green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=d94d06d17deb
I did a simple RTSP test on flame-kk which worked fine. We should remove such workarounds which obscure the code.
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Attachment #8620849 - Flags: review?(cpearce)
Attachment #8620849 - Flags: review?(cpearce) → review+
Thanks for the review.
You need to log in before you can comment on or make changes to this bug.