commit f0cc814343c167048edfbc050364c9195ad63e2e
author Danil Chapovalov <danilchap@webrtc.org> Tue Oct 31 17:59:39 2017 +0100
committer Commit Bot <commit-bot@chromium.org> Wed Nov 01 10:15:56 2017 +0000
tree 94d0b43a1fbb70fc670abe04dc735c8a31e99ae2
parent c4a14324e29cd1210cfa5d0c061e7a1f10dacf00

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 <ilnik@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20524}

modules/rtp_rtcp/source/rtp_header_extensions.cc

diff --git a/modules/rtp_rtcp/source/rtp_header_extensions.cc b/modules/rtp_rtcp/source/rtp_header_extensions.cc
index 3c6f3b6..958ad02 100644
--- a/modules/rtp_rtcp/source/rtp_header_extensions.cc
+++ b/modules/rtp_rtcp/source/rtp_header_extensions.cc
@@ -319,16 +319,18 @@
       data + VideoSendTiming::kPacerExitDeltaOffset,
       timing.pacer_exit_delta_ms);
   ByteWriter<uint16_t>::WriteBigEndian(
-      data + VideoSendTiming::kNetworkTimestampDeltaOffset, 0);  // reserved
+      data + VideoSendTiming::kNetworkTimestampDeltaOffset,
+      timing.network_timestamp_delta_ms);
   ByteWriter<uint16_t>::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<uint16_t>::WriteBigEndian(data + offset, time_delta_ms);
   return true;
 }