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

Unified Diff: webrtc/modules/rtp_rtcp/source/rtcp_utility.cc

Issue 1338203003: Wire up send-side bandwidth estimation. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/rtp_rtcp/source/rtcp_utility.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtcp_utility.cc b/webrtc/modules/rtp_rtcp/source/rtcp_utility.cc
index 4e37cf3716d916c7ecb1bfd6185b1790dfa15f5f..d94069a0188fc8464749da7f845f9d32d565429f 100644
--- a/webrtc/modules/rtp_rtcp/source/rtcp_utility.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtcp_utility.cc
@@ -9,6 +9,7 @@
*/
#include "webrtc/modules/rtp_rtcp/source/rtcp_utility.h"
+#include "webrtc/modules/rtp_rtcp/source/rtcp_packet/transport_feedback.h"
#include <assert.h>
#include <math.h> // ceil
@@ -76,6 +77,9 @@ RTCPUtility::RTCPParserV2::Packet() const
return _packet;
}
+rtcp::RtcpPacket* RTCPUtility::RTCPParserV2::ReleaseRtcpPacket() {
+ return rtcp_packet_.release();
+}
RTCPUtility::RTCPPacketTypes
RTCPUtility::RTCPParserV2::Begin()
{
@@ -1217,6 +1221,18 @@ RTCPUtility::RTCPParserV2::ParseFBCommon(const RTCPCommonHeader& header)
// Note: No state transition, SR REQ is empty!
stefan-webrtc 2015/09/17 10:33:19 May want to add a comment similar to this below as
sprang_webrtc 2015/09/17 15:08:28 Done.
return true;
}
+ case 15: {
+ _packetType = RTCPPacketTypes::kTransportFeedback;
+ rtcp_packet_ =
+ rtcp::TransportFeedback::ParseFrom(_ptrRTCPData - 12, length);
+ if (rtcp_packet_.get() == nullptr) {
+ EndCurrentBlock();
+ return false;
+ }
+
+ _ptrRTCPData += rtcp_packet_->BlockLength();
+ return true;
+ }
default:
break;
}

Powered by Google App Engine
This is Rietveld 408576698