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

Unified Diff: webrtc/media/engine/webrtcvideoengine2.cc

Issue 2437503004: Set actual transport overhead in rtp_rtcp (Closed)
Patch Set: Response to comments of honghaiz3 Created 4 years, 1 month 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/media/engine/webrtcvideoengine2.cc
diff --git a/webrtc/media/engine/webrtcvideoengine2.cc b/webrtc/media/engine/webrtcvideoengine2.cc
index 4d2d4cba08766fa62d31cddc0a6257fbc847830b..f58df35034eac64f1ae657faefe4946e4f0973bf 100644
--- a/webrtc/media/engine/webrtcvideoengine2.cc
+++ b/webrtc/media/engine/webrtcvideoengine2.cc
@@ -1524,6 +1524,12 @@ void WebRtcVideoChannel2::OnNetworkRouteChanged(
call_->OnNetworkRouteChanged(transport_name, network_route);
}
+void WebRtcVideoChannel2::OnTransportOverheadChange(
+ int transport_overhead_per_packet) {
+ call_->SignalTransportOverheadChange(webrtc::MediaType::VIDEO,
stefan-webrtc 2016/11/02 13:21:43 Why SignalTransportOverheadChange instead of OnTra
michaelt 2016/11/02 13:35:34 SignalNetworkState was the prototype for this func
stefan-webrtc 2016/11/02 13:41:09 I'd prefer On to align with the other functions in
michaelt 2016/11/02 15:13:38 Done.
+ transport_overhead_per_packet);
+}
+
void WebRtcVideoChannel2::SetInterface(NetworkInterface* iface) {
MediaChannel::SetInterface(iface);
// Set the RTP recv/send buffer to a bigger size

Powered by Google App Engine
This is Rietveld 408576698