zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove redundant RTC_DCHECK of max/min RTP header extension id

Browse Source 
This RTC_DCHECK is no longer valid when we start sending two-byte header
extensions.

Bug: webrtc:7990
Change-Id: Id6cda6f3647838a0d93aa48528bb0bb4300304d3
Reviewed-on: https://webrtc-review.googlesource.com/c/105104
Reviewed-by: Stefan Holmer <stefan@webrtc.org>
Commit-Queue: Johannes Kron <kron@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25315}
This commit is contained in:
Johannes Kron
2018-10-10 16:54:46 +02:00
committed by Commit Bot
parent 93922dca97
commit 362cb50f92
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
call/rtp_video_sender.cc
View File

@ -257,9 +257,6 @@ RtpVideoSender::RtpVideoSender(
for (size_t i = 0; i < rtp_config_.extensions.size(); ++i) {
const std::string& extension = rtp_config_.extensions[i].uri;
int id = rtp_config_.extensions[i].id;
// One-byte-extension local identifiers are in the range 1-14 inclusive.
RTC_DCHECK_GE(id, 1);
RTC_DCHECK_LE(id, 14);
RTC_DCHECK(RtpExtension::IsSupportedForVideo(extension));
for (auto& rtp_rtcp : rtp_modules_) {
RTC_CHECK(rtp_rtcp->RegisterRtpHeaderExtension(extension, id));