Rename flexfec AddAndProcessReceivedPacket --> OnRtpPacket.

webrtc/call/flexfec_receive_stream_impl.cc

Index: webrtc/call/flexfec_receive_stream_impl.cc
diff --git a/webrtc/call/flexfec_receive_stream_impl.cc b/webrtc/call/flexfec_receive_stream_impl.cc
index f5272c9029dd48f2e8ba77866a0df0c81fe81b1c..02e0f12fe76137e0c384f44d39d8f9e6c484d142 100644
--- a/webrtc/call/flexfec_receive_stream_impl.cc
+++ b/webrtc/call/flexfec_receive_stream_impl.cc
@@ -149,19 +149,18 @@ FlexfecReceiveStreamImpl::~FlexfecReceiveStreamImpl() {
   process_thread_->DeRegisterModule(rtp_rtcp_.get());
 }
 
-bool FlexfecReceiveStreamImpl::AddAndProcessReceivedPacket(
-    const RtpPacketReceived& packet) {
+void FlexfecReceiveStreamImpl::OnRtpPacket(const RtpPacketReceived& packet) {
   {
     rtc::CritScope cs(&crit_);
     if (!started_)
-      return false;
+      return;
   }
 
   if (!receiver_)
-    return false;
+    return;
 
   if (!receiver_->AddAndProcessReceivedPacket(packet))

[nisse-webrtc, 2017/02/10 13:46:21]

Should we rename and/or drop the return value for this method too?

Is it essential to exclude broken flexfec packets from the below statistics?

[brandtr, 2017/02/10 14:12:21]

Yes, might as well do both :)
Don't think so. What change do you want to make?

[nisse-webrtc, 2017/02/13 13:20:03]

If I delete return value and this if statement, as a consequence the below stats
call will be invoked also for invalid packets.

[brandtr, 2017/02/13 15:32:13]

I understand. I think it's fine to include those.

-    return false;
+    return;
 
   // Do not report media packets in the RTCP RRs generated by |rtp_rtcp_|.
   if (packet.Ssrc() == config_.remote_ssrc) {
@@ -172,8 +171,6 @@ bool FlexfecReceiveStreamImpl::AddAndProcessReceivedPacket(
     rtp_receive_statistics_->IncomingPacket(header, packet.size(),
                                             kNotRetransmitted);
   }
-
-  return true;
 }
 
 void FlexfecReceiveStreamImpl::Start() {