Fix for multiple nal units per frame
Dynamic framerate exposed a bug in OMX as the algorithm started producing
frames with multiple NAL units. OMX was tagging correct time stamp only
to last NAL unit in each frame. This has been fixed now to tag the same
timestamp to all NAL units belonging to each frame. Looks like PV muxer
and demuxer are compatible and apparently this is not an issue for them.
But quicktime and VLC don't like NAL units with 0 timestamp in the
middle of a bitstream.
Originally from: https://partner.source.android.com/g/#change,1402
diff --git a/omx/video/src/openmax_il/video_encode/src/OMX_VideoEnc_Utils.c b/omx/video/src/openmax_il/video_encode/src/OMX_VideoEnc_Utils.c
index e62de65..54aaf9f 100644
--- a/omx/video/src/openmax_il/video_encode/src/OMX_VideoEnc_Utils.c
+++ b/omx/video/src/openmax_il/video_encode/src/OMX_VideoEnc_Utils.c
@@ -2789,9 +2789,9 @@
pBufHead->pMarkData = NULL;
pBufHead->hMarkTargetComponent = NULL;
- pBufHead->nTickCount = 0;
- pBufHead->nTimeStamp = 0;
pBufHead->nFlags = 0;
+ pBufHead->nTickCount = pComponentPrivate->sCircularBuffer.pHead->nTickCount;
+ pBufHead->nTimeStamp = pComponentPrivate->sCircularBuffer.pHead->nTimeStamp;
}
else
{
