webrtc::RtpPacket name freed for better RtpPacket

webrtc/modules/rtp_rtcp/source/fec_test_helper.h

Index: webrtc/modules/rtp_rtcp/source/fec_test_helper.h
diff --git a/webrtc/modules/rtp_rtcp/source/fec_test_helper.h b/webrtc/modules/rtp_rtcp/source/fec_test_helper.h
index aacc2d1ecc349ddef7ee141ca16f333257d545bb..df7075ac5c844227ab668d5327540757b431e4fa 100644
--- a/webrtc/modules/rtp_rtcp/source/fec_test_helper.h
+++ b/webrtc/modules/rtp_rtcp/source/fec_test_helper.h
@@ -20,9 +20,7 @@ const uint8_t kFecPayloadType = 96;
 const uint8_t kRedPayloadType = 97;
 const uint8_t kVp8PayloadType = 120;
 
-typedef ForwardErrorCorrection::Packet Packet;
-
-struct RtpPacket : public Packet {
+struct FecRtpPacket : public ForwardErrorCorrection::Packet {

[stefan-webrtc, 2016/02/19 14:21:15]

Could you define this within the test namespace instead? Now it sounds like this
is a packet containing fec data, which it's not. 

[danilchap, 2016/02/19 14:48:29]

checked what it containing and renamed to test::RawRtpPacket.

[stefan-webrtc, 2016/02/19 14:56:21]

Ok. test::RtpPacket would work too I guess? Doesn't really matter, but shorter.

   WebRtcRTPHeader header;
 };
 
@@ -34,17 +32,18 @@ class FrameGenerator {
 
   uint16_t NextSeqNum();
 
-  RtpPacket* NextPacket(int offset, size_t length);
+  FecRtpPacket* NextPacket(int offset, size_t length);
 
   // Creates a new RtpPacket with the RED header added to the packet.
-  RtpPacket* BuildMediaRedPacket(const RtpPacket* packet);
+  FecRtpPacket* BuildMediaRedPacket(const FecRtpPacket* packet);
 
   // Creates a new RtpPacket with FEC payload and red header. Does this by
   // creating a new fake media RtpPacket, clears the marker bit and adds a RED
   // header. Finally replaces the payload with the content of |packet->data|.
-  RtpPacket* BuildFecRedPacket(const Packet* packet);
+  FecRtpPacket* BuildFecRedPacket(const ForwardErrorCorrection::Packet* packet);
 
-  void SetRedHeader(Packet* red_packet, uint8_t payload_type,
+  void SetRedHeader(ForwardErrorCorrection::Packet* red_packet,
+                    uint8_t payload_type,
                     size_t header_length) const;
 
  private: