Move RTP/RTCP demuxing logic from BaseChannel to RtpTransport.

webrtc/media/base/rtputils.h

Index: webrtc/media/base/rtputils.h
diff --git a/webrtc/media/base/rtputils.h b/webrtc/media/base/rtputils.h
index 1a710576117e72debbdfb925ec3a225d8afcd9ed..03c21276636c0820c4664c9f3e9d15c5aab5e271 100644
--- a/webrtc/media/base/rtputils.h
+++ b/webrtc/media/base/rtputils.h
@@ -58,6 +58,10 @@ bool IsRtpPacket(const void* data, size_t len);
 // True if |payload type| is 0-127.
 bool IsValidRtpPayloadType(int payload_type);
 
+bool IsValidRtpRtcpPacketSize(bool rtcp, size_t size);
+
+const char* PacketType(bool rtcp);

[Taylor Brandstetter, 2017/05/25 16:14:25]

Can you add a comment above these declarations? And if we don't have one
somewhere else, can you add a TODO to use an enum to differentiate RTP/RTCP
instead of a bool?

Also, since PacketType has a larger scope now, a more descriptive name would be
good.

[Zach Stein, 2017/05/30 21:50:55]

Done.

+
 // Verifies that a packet has a valid RTP header.
 bool ValidateRtpHeader(const uint8_t* rtp,
                        size_t length,