commit f25761d798f3639900014be326270e0475c640cd
author Tomas Gunnarsson <tommi@webrtc.org> Wed Jun 03 20:55:33 2020
committer Commit Bot <commit-bot@chromium.org> Thu Jun 04 08:11:21 2020
tree 670bb31c1b97b797efe1e88dfd649db7cedcad01
parent 4d75d9c04c39ace96d07de7fb6425b48fb873357

Remove dependency from RtpRtcp on the Module interface.

The 'Module' part of the implementation must not be
called via the RtpRtcp interface, but is rather a part of
the contract with ProcessThread. That in turn is an
implementation detail for how timers are currently implemented
in the default implementation.

Along the way I'm deprecating away the factory function which
was inside the interface and tied it to one specific implementation.
Instead, I'm moving that to the implementation itself and down the
line, we don't have to go through it if we just want to create an
instance of the class.

The key change is in rtp_rtcp.h and the new rtp_rtcp_interface.h
header file (things moved from rtp_rtcp.h), the rest falls from that.

Change-Id: I294f13e947b9e3e4e649400ee94a11a81e8071ce
Bug: webrtc:11581
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/176419
Reviewed-by: Magnus Flodman <mflodman@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31440}