Tool to convert RtcEventLog files to RtpDump format.

webrtc/test/rtp_file_writer.cc

Index: webrtc/test/rtp_file_writer.cc
diff --git a/webrtc/test/rtp_file_writer.cc b/webrtc/test/rtp_file_writer.cc
index 793e51a55eea95553dd54e09a096e115eddfcd31..d9e058646814a0dc80b75b8af0159521f20ab167 100644
--- a/webrtc/test/rtp_file_writer.cc
+++ b/webrtc/test/rtp_file_writer.cc
@@ -40,7 +40,6 @@ class RtpDumpWriter : public RtpFileWriter {
 
   bool WritePacket(const RtpPacket* packet) override {
     uint16_t len = static_cast<uint16_t>(packet->length + kPacketHeaderSize);
-    RTC_CHECK_GE(packet->original_length, packet->length);

[kjellander_webrtc, 2015/09/17 14:49:36]

Can you explain why this check needs to go away?

[ivoc, 2015/09/17 14:53:29]

Of course, I added support for writing RTCP packets in the RTPdump format, and
for RTCP packets the original_length should be set to 0 (see for example the
plen field: http://www.cs.columbia.edu/irt/software/rtptools/#rtpdump ). So this
CHECK is preventing RTCP packets from being written.

[kjellander_webrtc, 2015/09/18 05:41:18]

Thanks for the explanation!

     uint16_t plen = static_cast<uint16_t>(packet->original_length);
     uint32_t offset = packet->time_ms;
     RTC_CHECK(WriteUint16(len));