Avoid multiple calls to webrtc::field_trial::FindFullName in RTPSender

webrtc/modules/rtp_rtcp/source/rtp_sender.cc

Index: webrtc/modules/rtp_rtcp/source/rtp_sender.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
index ae491209b7f22edf1ff3d063a66ded37dfeff200..add7c21e9be22e109c15227de2b7ee0ac4336c7f 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
@@ -123,7 +123,10 @@ RTPSender::RTPSender(
       rtx_(kRtxOff),
       rtp_overhead_bytes_per_packet_(0),
       retransmission_rate_limiter_(retransmission_rate_limiter),
-      overhead_observer_(overhead_observer) {
+      overhead_observer_(overhead_observer),
+      send_side_bwe_with_overhead_(
+          webrtc::field_trial::FindFullName(
+              "WebRTC-SendSideBwe-WithOverhead") == "Enabled") {
   ssrc_ = ssrc_db_->CreateSSRC();
   RTC_DCHECK(ssrc_ != 0);
   ssrc_rtx_ = ssrc_db_->CreateSSRC();
@@ -1256,8 +1259,7 @@ void RTPSender::AddPacketToTransportFeedback(uint16_t packet_id,
                                              const RtpPacketToSend& packet,
                                              int probe_cluster_id) {
   size_t packet_size = packet.payload_size() + packet.padding_size();
-  if (webrtc::field_trial::FindFullName("WebRTC-SendSideBwe-WithOverhead") ==

[minyue-webrtc, 2017/01/18 08:14:38]

but it is called only once, isn't it?

[stefan-webrtc, 2017/01/18 08:41:40]

Once for every packet sent. Still a lot... :)

-      "Enabled") {
+  if (send_side_bwe_with_overhead_) {
     packet_size = packet.size();
   }