Generalize FEC header formatting. (pt. 4)

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 e2717d3432ecfd2bee3fc3be9865d7ed8a794143..6f2aea45690ca477797f4467b82708b6ca296948 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
@@ -342,7 +342,7 @@ size_t RTPSender::MaxDataPayloadLength() const {
     return max_payload_length_ - RtpHeaderLength();
   } else {
     return max_payload_length_ - RtpHeaderLength()  // RTP overhead.
-           - video_->FECPacketOverhead()            // FEC/ULP/RED overhead.
+           - video_->FecPacketOverhead()            // FEC/ULP/RED overhead.

[danilchap, 2016/08/22 13:00:41]

Non-functional renames, adding/removing empty lines, fixing comments, other
inline non-functional changes easier to review when they are in separate CL.

It is easier to jump between files in CL when there are less of them changed.

[brandtr, 2016/08/23 08:19:11]

Done, see
https://codereview.webrtc.org/2275443002/diff/1/webrtc/modules/rtp_rtcp/sourc....

            - (RtxStatus() ? kRtxHeaderSize : 0);    // RTX overhead.
   }
 }