Revert of Refactoring of RTCP options in BaseChannel.

Revert of Refactoring of RTCP options in BaseChannel. (patchset #4 id:60001 of https://codereview.webrtc.org/2622613004/ )

Reason for revert:
Broke Mac64 Release [large tests] build
https://build.chromium.org/p/client.webrtc/builders/Mac64%20Release%20%5Blarge%20tests%5D/builds/10576

Original issue's description:
> Refactoring of RTCP options in BaseChannel.
> 
> Previously, BaseChannel supported a "no RTCP" mode, which wasn't
> being used any more and is being deleted.
> 
> Also, "RTCP mux required" previously worked by calling "ActivateRtcpMux"
> after construction. Now it works by explicitly passing a
> "require_rtcp_mux" parameter into the constructor.
> 
> BUG=None
> 
> Review-Url: https://codereview.webrtc.org/2622613004
> Cr-Commit-Position: refs/heads/master@{#16045}
> Committed: https://chromium.googlesource.com/external/webrtc/+/ac22f70906175831270b399e20d0519198ac6922

TBR=pthatcher@webrtc.org,deadbeef@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None

[keishi, 2017-01-13 06:29:49]

The CQ bit was checked by keishi@chromium.org

[keishi, 2017-01-13 06:29:49]

Created Revert of Refactoring of RTCP options in BaseChannel.

[commit-bot: I haz the power, 2017-01-13 06:29:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-01-13 06:29:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-13 06:30:00]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-webrtc-committers".
Note that this has nothing to do with OWNERS files.

[Taylor Brandstetter, 2017-01-13 07:16:34]

On 2017/01/13 06:30:00, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. 
> CQ run can only be started by full committers or once the patch has
> received an L-G-T-M from a full committer.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> Committers are members of the group "project-webrtc-committers".
> Note that this has nothing to do with OWNERS files.

Are you sure that this CL could have caused the problem? It looks like it's
failing to find "/Users/chrome-bot/goma/gomacc", and this CL doesn't touch any
build files.

[keishi, 2017-01-13 07:18:31]

On 2017/01/13 07:16:34, Taylor Brandstetter wrote:
> On 2017/01/13 06:30:00, commit-bot: I haz the power wrote:
> > No L-G-T-M from a valid reviewer yet. 
> > CQ run can only be started by full committers or once the patch has
> > received an L-G-T-M from a full committer.
> > Even if an L-G-T-M may have been provided, it was from a non-committer,
> > _not_ a full super star committer.
> > Committers are members of the group "project-webrtc-committers".
> > Note that this has nothing to do with OWNERS files.
> 
> Are you sure that this CL could have caused the problem? It looks like it's
> failing to find "/Users/chrome-bot/goma/gomacc", and this CL doesn't touch any
> build files.

Sorry it was the only one in the regression range. Closing