From f0cc814343c167048edfbc050364c9195ad63e2e Mon Sep 17 00:00:00 2001 From: Danil Chapovalov Date: Tue, 31 Oct 2017 17:59:39 +0100 Subject: [PATCH] Support writing network timestamp delta fields into VideoTimingExtension Bug: None Change-Id: I17b9ba0eb8095cfd8e6bc5bf97b2949d5d3edd24 Reviewed-on: https://webrtc-review.googlesource.com/17500 Reviewed-by: Ilya Nikolaevskiy Commit-Queue: Danil Chapovalov Cr-Commit-Position: refs/heads/master@{#20524} --- modules/rtp_rtcp/source/rtp_header_extensions.cc | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/modules/rtp_rtcp/source/rtp_header_extensions.cc b/modules/rtp_rtcp/source/rtp_header_extensions.cc index 3c6f3b6341..958ad028a9 100644 --- a/modules/rtp_rtcp/source/rtp_header_extensions.cc +++ b/modules/rtp_rtcp/source/rtp_header_extensions.cc @@ -319,16 +319,18 @@ bool VideoTimingExtension::Write(uint8_t* data, const VideoSendTiming& timing) { data + VideoSendTiming::kPacerExitDeltaOffset, timing.pacer_exit_delta_ms); ByteWriter::WriteBigEndian( - data + VideoSendTiming::kNetworkTimestampDeltaOffset, 0); // reserved + data + VideoSendTiming::kNetworkTimestampDeltaOffset, + timing.network_timestamp_delta_ms); ByteWriter::WriteBigEndian( - data + VideoSendTiming::kNetwork2TimestampDeltaOffset, 0); // reserved + data + VideoSendTiming::kNetwork2TimestampDeltaOffset, + timing.network2_timestamp_delta_ms); return true; } bool VideoTimingExtension::Write(uint8_t* data, uint16_t time_delta_ms, uint8_t offset) { - RTC_DCHECK_LT(offset, kValueSizeBytes - sizeof(uint16_t)); + RTC_DCHECK_LE(offset, kValueSizeBytes - sizeof(uint16_t)); ByteWriter::WriteBigEndian(data + offset, time_delta_ms); return true; }