Set actual transport overhead in rtp_rtcp

webrtc/media/engine/webrtcvideoengine2.cc

 /*
  *  Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 1506 matching lines @@
       webrtc::MediaType::VIDEO,
       ready ? webrtc::kNetworkUp : webrtc::kNetworkDown);
 }
 
 void WebRtcVideoChannel2::OnNetworkRouteChanged(
     const std::string& transport_name,
     const rtc::NetworkRoute& network_route) {
   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 OnTransportOverheadChange?

[michaelt, 2016/11/02 13:35:34]

SignalNetworkState was the prototype for this function. But beside that there is
no reason why I called it "SignalTransportOverheadChange" and not
"OnTransportOverheadChange".

[stefan-webrtc, 2016/11/02 13:41:09]

I'd prefer On to align with the other functions in this chain of calls.

[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
   MediaChannel::SetOption(NetworkInterface::ST_RTP,
                           rtc::Socket::OPT_RCVBUF,
                           kVideoRtpBufferSize);
 
   // Speculative change to increase the outbound socket buffer size.
   // In b/15152257, we are seeing a significant number of packets discarded
   // due to lack of socket buffer space, although it's not yet clear what the
@@ skipping 1155 matching lines @@
         rtx_mapping[video_codecs[i].codec.id] !=
             ulpfec_config.red_payload_type) {
       video_codecs[i].rtx_payload_type = rtx_mapping[video_codecs[i].codec.id];
     }
   }
 
   return video_codecs;
 }
 
 }  // namespace cricket