Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(53)

Side by Side Diff: webrtc/video/rtp_stream_receiver.cc

Issue 1984983002: Remove use of RtpHeaderExtension and clean up (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Addressed nit Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 120
121 rtp_rtcp_->SetRTCPStatus(config.rtp.rtcp_mode); 121 rtp_rtcp_->SetRTCPStatus(config.rtp.rtcp_mode);
122 rtp_rtcp_->SetSSRC(config.rtp.local_ssrc); 122 rtp_rtcp_->SetSSRC(config.rtp.local_ssrc);
123 rtp_rtcp_->SetKeyFrameRequestMethod(kKeyFrameReqPliRtcp); 123 rtp_rtcp_->SetKeyFrameRequestMethod(kKeyFrameReqPliRtcp);
124 if (config.rtp.remb) { 124 if (config.rtp.remb) {
125 rtp_rtcp_->SetREMBStatus(true); 125 rtp_rtcp_->SetREMBStatus(true);
126 remb_->AddReceiveChannel(rtp_rtcp_.get()); 126 remb_->AddReceiveChannel(rtp_rtcp_.get());
127 } 127 }
128 128
129 for (size_t i = 0; i < config.rtp.extensions.size(); ++i) { 129 for (size_t i = 0; i < config.rtp.extensions.size(); ++i) {
130 EnableReceiveRtpHeaderExtension(config.rtp.extensions[i].name, 130 EnableReceiveRtpHeaderExtension(config.rtp.extensions[i].uri,
131 config.rtp.extensions[i].id); 131 config.rtp.extensions[i].id);
132 } 132 }
133 133
134 static const int kMaxPacketAgeToNack = 450; 134 static const int kMaxPacketAgeToNack = 450;
135 const int max_reordering_threshold = (config.rtp.nack.rtp_history_ms > 0) 135 const int max_reordering_threshold = (config.rtp.nack.rtp_history_ms > 0)
136 ? kMaxPacketAgeToNack : kDefaultMaxReorderingThreshold; 136 ? kMaxPacketAgeToNack : kDefaultMaxReorderingThreshold;
137 rtp_receive_statistics_->SetMaxReorderingThreshold(max_reordering_threshold); 137 rtp_receive_statistics_->SetMaxReorderingThreshold(max_reordering_threshold);
138 138
139 // TODO(pbos): Support multiple RTX, per video payload. 139 // TODO(pbos): Support multiple RTX, per video payload.
140 for (const auto& kv : config_.rtp.rtx) { 140 for (const auto& kv : config_.rtp.rtx) {
(...skipping 392 matching lines...) Expand 10 before | Expand all | Expand 10 after
533 const std::string& extension, int id) { 533 const std::string& extension, int id) {
534 // One-byte-extension local identifiers are in the range 1-14 inclusive. 534 // One-byte-extension local identifiers are in the range 1-14 inclusive.
535 RTC_DCHECK_GE(id, 1); 535 RTC_DCHECK_GE(id, 1);
536 RTC_DCHECK_LE(id, 14); 536 RTC_DCHECK_LE(id, 14);
537 RTC_DCHECK(RtpExtension::IsSupportedForVideo(extension)); 537 RTC_DCHECK(RtpExtension::IsSupportedForVideo(extension));
538 RTC_CHECK(rtp_header_parser_->RegisterRtpHeaderExtension( 538 RTC_CHECK(rtp_header_parser_->RegisterRtpHeaderExtension(
539 StringToRtpExtensionType(extension), id)); 539 StringToRtpExtensionType(extension), id));
540 } 540 }
541 541
542 } // namespace webrtc 542 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698