Closed Bug 1497610 Opened Last year Closed Last year
Newer Or Same Timestamp
47 bytes, text/x-phabricator-request
|Details | Review|
Upstream just uses IsNewerTimestamp. We should research why we made this change, and if it is still relevant, attempt to upstream it.
Try run for the removal here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b6e1b30de38612a5d44e70daae8a769a3247a481
So this was from bug 985254, which made the H264 packetization code work reasonably correctly. There's no way a Try run would catch any issues with that code. This was so that it wouldn't barf if we passed single SPS or PPS NALs in with the same timestamp as a later Iframe NAL -- in RTP, multiple NALs with the same timestamp are ok, it's only wrong if the timestamp goes backwards. So while much of the code we landed in bug 985254 may not be relevant (or if relevant may require recoding), this is likely still correct, even if it currently avoids tripping on this problem due to other processing in the reception of H264 and de-jittering.
You can see an earlier attempt to upstream this code, linked to by that bug
Similar to Bug 1497974, these are only used in VCMJitterBuffer, which the older jitter buffer that we no longer use. We're safe to remove these.
The affected functions are only used by VCMJitterBuffer, which is the older jitter buffer that is no longer in use. We can safely remove these modifications.
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/5ed0ed11b23d Remove IsNewerOrSameTimestamp; r=bwc
You need to log in before you can comment on or make changes to this bug.