Move bundle filter from BaseChannel to RtpTransport.

Move bundle filter from BaseChannel to RtpTransport.

Subsumed by https://codereview.chromium.org/2890263003/

BUG=webrtc:7013

[Zach Stein, 2017-05-15 22:44:01]

zstein@webrtc.org changed reviewers:
+ deadbeef@webrtc.org

[Zach Stein, 2017-05-15 22:44:01]

Not sure if this is the right approach or not - let me know what you think.

[zhihuang1, 2017-05-16 17:59:01]

On 2017/05/15 22:44:01, Zach Stein wrote:
> Not sure if this is the right approach or not - let me know what you think.

Looks good to me. Though I don't have power here.

[Taylor Brandstetter, 2017-05-17 02:12:42]

This looks good, but moving BundleFilter to RtpTransport doesn't serve much
purpose in itself; could this CL also move the "received packet" logic to
RtpTransport? In other words, change the CL from "move BundleFilter" to "move
demuxing responsibility".

How I'd imagine this looking is that instead of BaseChannel hooking up to
"SignalReadPacket" directly, RtpTransport would hook up to that signal, and pass
along the packet to BaseChannel when the packet is demuxed. Remember one of the
end goals is to decouple BaseChannel from
PacketTransportInterface/DtlsTransportInternal.

Also, we may want to use a callback interface like in this CL:
https://codereview.webrtc.org/2867943003/diff/120001/webrtc/call/rtp_demuxer.cc

... since in the future ("phase 6" in our plan?), one RtpTransport will be used
by multiple BaseChannels.

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/channel.h
File webrtc/pc/channel.h (right):

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/channel.h#newcode200
webrtc/pc/channel.h:200: bool HandlesPayloadType(int payload_type);
const?

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/rtptransport.cc
File webrtc/pc/rtptransport.cc (right):

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/rtptransport.cc#new...
webrtc/pc/rtptransport.cc:100: }
nit: newline

[Zach Stein, 2017-05-30 19:08:38]

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/channel.h
File webrtc/pc/channel.h (right):

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/channel.h#newcode200
webrtc/pc/channel.h:200: bool HandlesPayloadType(int payload_type);
On 2017/05/17 02:12:42, Taylor Brandstetter wrote:
> const?

Done.

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/rtptransport.cc
File webrtc/pc/rtptransport.cc (right):

https://codereview.webrtc.org/2889453002/diff/1/webrtc/pc/rtptransport.cc#new...
webrtc/pc/rtptransport.cc:100: }
On 2017/05/17 02:12:42, Taylor Brandstetter wrote:
> nit: newline

Done.

[Zach Stein, 2017-05-30 21:53:40]

Description was changed from

==========
Move bundle filter from BaseChannel to RtpTransport.

BUG=webrtc:7013
==========

to

==========
Move bundle filter from BaseChannel to RtpTransport.

Subsumed by https://codereview.chromium.org/2890263003/

BUG=webrtc:7013
==========