From 603ce9835b8ecbe0e716d685753d9815021f6d0f Mon Sep 17 00:00:00 2001 From: Danil Chapovalov Date: Wed, 27 Dec 2017 11:32:50 +0100 Subject: [PATCH] Reenable the dcheck ssrc can't change after first sent packet MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Bug: webrtc:6887 Change-Id: I6eb1ffc7dd98390f870b15132ba6038dd6c57b87 Reviewed-on: https://webrtc-review.googlesource.com/36301 Reviewed-by: Erik Språng Commit-Queue: Danil Chapovalov Cr-Commit-Position: refs/heads/master@{#21453} --- modules/rtp_rtcp/source/rtp_sender.cc | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/modules/rtp_rtcp/source/rtp_sender.cc b/modules/rtp_rtcp/source/rtp_sender.cc index bc880393ac..9ee5f3ba09 100644 --- a/modules/rtp_rtcp/source/rtp_sender.cc +++ b/modules/rtp_rtcp/source/rtp_sender.cc @@ -927,10 +927,7 @@ bool RTPSender::SendToNetwork(std::unique_ptr packet, // To support retransmissions, we store the media packet as sent in the // packet history (even if send failed). if (storage == kAllowRetransmission) { - // TODO(brandtr): Uncomment the DCHECK line below when |ssrc_| cannot - // change after the first packet has been sent. For more details, see - // https://bugs.chromium.org/p/webrtc/issues/detail?id=6887. - // RTC_DCHECK_EQ(ssrc, SSRC()); + RTC_DCHECK_EQ(ssrc, SSRC()); packet_history_.PutRtpPacket(std::move(packet), storage, true); }