Make Call::OnRecoveredPacket parse RTP header and call OnRtpPacket.

webrtc/modules/rtp_rtcp/source/rtp_packet_received.h

Index: webrtc/modules/rtp_rtcp/source/rtp_packet_received.h
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_packet_received.h b/webrtc/modules/rtp_rtcp/source/rtp_packet_received.h
index 95674cf863f1265ee8ebb3c54a2978114339faf3..7eba81523e25a33295c3f10113f0844c43105f5a 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_packet_received.h
+++ b/webrtc/modules/rtp_rtcp/source/rtp_packet_received.h
@@ -40,6 +40,10 @@ class RtpPacketReceived : public rtp::Packet {
   bool retransmit() const { return retransmit_; }

[brandtr, 2017/02/15 12:20:50]

rename to 'retransmitted'/'set_retransmitted', to be consistent with the new
methods below.

[nisse-webrtc, 2017/02/15 13:59:14]

On the receive side, that verb doesn't seem quite right to me. *We* haven't
retransmitted anything; it's just that we received the packet as an rtx payload,
supposedly because the other end retransmitted it. 

[nisse-webrtc, 2017/05/09 12:37:56]

Danil suggested that we use only a single "recovered" flag, and use it for both
rtx and fec. So I'll delete retransmit. 

   void set_retransmit(bool value) { retransmit_ = value; }
 
+  // Flag if packet was recovered via fec.
+  bool recovered() const { return recovered_; }
+  void set_recovered(bool value) { recovered_ = value; }
+
   int payload_type_frequency() const { return payload_type_frequency_; }
   void set_payload_type_frequency(int value) {
     payload_type_frequency_ = value;
@@ -50,6 +54,7 @@ class RtpPacketReceived : public rtp::Packet {
   int64_t arrival_time_ms_ = 0;
   int payload_type_frequency_ = 0;
   bool retransmit_ = false;
+  bool recovered_ = false;
 };
 
 }  // namespace webrtc