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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/rtcp_sender.cc

Issue 2540363003: Wire up rtcp xr target bitrate on receive side. (Closed)
Patch Set: Actual fuzzer fix Created 4 years 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 946 matching lines...) Expand 10 before | Expand all | Expand 10 after
957 957
958 SetFlag(kRtcpApp, true); 958 SetFlag(kRtcpApp, true);
959 app_sub_type_ = subType; 959 app_sub_type_ = subType;
960 app_name_ = name; 960 app_name_ = name;
961 app_data_.reset(new uint8_t[length]); 961 app_data_.reset(new uint8_t[length]);
962 app_length_ = length; 962 app_length_ = length;
963 memcpy(app_data_.get(), data, length); 963 memcpy(app_data_.get(), data, length);
964 return 0; 964 return 0;
965 } 965 }
966 966
967 // TODO(sprang): Remove support for VoIP metrics? (Not used in receiver.)
967 int32_t RTCPSender::SetRTCPVoIPMetrics(const RTCPVoIPMetric* VoIPMetric) { 968 int32_t RTCPSender::SetRTCPVoIPMetrics(const RTCPVoIPMetric* VoIPMetric) {
968 rtc::CritScope lock(&critical_section_rtcp_sender_); 969 rtc::CritScope lock(&critical_section_rtcp_sender_);
969 xr_voip_metric_.emplace(*VoIPMetric); 970 xr_voip_metric_.emplace(*VoIPMetric);
970 971
971 SetFlag(kRtcpAnyExtendedReports, true); 972 SetFlag(kRtcpAnyExtendedReports, true);
972 return 0; 973 return 0;
973 } 974 }
974 975
975 void RTCPSender::SendRtcpXrReceiverReferenceTime(bool enable) { 976 void RTCPSender::SendRtcpXrReceiverReferenceTime(bool enable) {
976 rtc::CritScope lock(&critical_section_rtcp_sender_); 977 rtc::CritScope lock(&critical_section_rtcp_sender_);
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
1054 // but we can't because of an incorrect warning (C4822) in MVS 2013. 1055 // but we can't because of an incorrect warning (C4822) in MVS 2013.
1055 } sender(transport_, event_log_); 1056 } sender(transport_, event_log_);
1056 1057
1057 RTC_DCHECK_LE(max_payload_length_, IP_PACKET_SIZE); 1058 RTC_DCHECK_LE(max_payload_length_, IP_PACKET_SIZE);
1058 uint8_t buffer[IP_PACKET_SIZE]; 1059 uint8_t buffer[IP_PACKET_SIZE];
1059 return packet.BuildExternalBuffer(buffer, max_payload_length_, &sender) && 1060 return packet.BuildExternalBuffer(buffer, max_payload_length_, &sender) &&
1060 !sender.send_failure_; 1061 !sender.send_failure_;
1061 } 1062 }
1062 1063
1063 } // namespace webrtc 1064 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtcp_receiver_unittest.cc ('k') | webrtc/modules/rtp_rtcp/source/rtp_rtcp_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698