Have AudioSendStream register CNG payload types with the RtpRtcpModule.

Have AudioSendStream register CNG payload types with the RtpRtcpModule.

TBR=kwiberg@webrtc.org # Turn perf-bots green again

BUG=webrtc:5806
Review-Url: https://codereview.webrtc.org/2844803003
Cr-Commit-Position: refs/heads/master@{#17911}
Committed: https://chromium.googlesource.com/external/webrtc/+/3b9ff38d8a10d46f69680c5634b4814fb02378e2

[ossu, 2017-04-27 13:51:47]

The CQ bit was checked by ossu@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-27 13:51:50]

Dry run: 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-04-27 14:02:00]

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

[commit-bot: I haz the power, 2017-04-27 14:02:01]

Dry run: Try jobs failed on following builders:
  linux_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_rel/builds/25556)

[ossu, 2017-04-27 14:25:02]

The CQ bit was checked by ossu@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-27 14:25:05]

Dry run: CQ is trying da patch.

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

[ossu, 2017-04-27 14:44:33]

ossu@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[ossu, 2017-04-27 14:44:33]

PTAL. The problem it fixes is turning some of our bots red, so I'll be tbr:ing
this. Feel free to pick it apart and I'll clean it up tomorrow.

Turns out CN types weren't registered properly when using the new SetEncoder
interface. Also turns out this mostly didn't matter, since Channel registers a
default set of codecs, including all the CN types, at creation. This came to
light because a Chromium perf-test was spewing error messages. The payload type
id at 16kHz (we were using it with ISAC) was different from the default set in
the ACMs codec database.

[commit-bot: I haz the power, 2017-04-27 14:59:31]

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

[commit-bot: I haz the power, 2017-04-27 14:59:32]

Dry run: This issue passed the CQ dry run.

[ossu, 2017-04-27 15:00:45]

Description was changed from

==========
Have AudioSendStream register CNG payload types with the RtpRtcpModule.

BUG=webrtc:5806
==========

to

==========
Have AudioSendStream register CNG payload types with the RtpRtcpModule.

TBR=kwiberg@webrtc.org # Turn perf-bots green again

BUG=webrtc:5806
==========

[ossu, 2017-04-27 15:00:50]

The CQ bit was checked by ossu@webrtc.org

[commit-bot: I haz the power, 2017-04-27 15:00:53]

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-04-27 15:03:43]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493305250729160,
"parent_rev": "9078d8cf05f6307db640509ab9950592d3956e20", "commit_rev":
"3b9ff38d8a10d46f69680c5634b4814fb02378e2"}

[commit-bot: I haz the power, 2017-04-27 15:03:46]

Description was changed from

==========
Have AudioSendStream register CNG payload types with the RtpRtcpModule.

TBR=kwiberg@webrtc.org # Turn perf-bots green again

BUG=webrtc:5806
==========

to

==========
Have AudioSendStream register CNG payload types with the RtpRtcpModule.

TBR=kwiberg@webrtc.org # Turn perf-bots green again

BUG=webrtc:5806

Review-Url: https://codereview.webrtc.org/2844803003
Cr-Commit-Position: refs/heads/master@{#17911}
Committed:
https://chromium.googlesource.com/external/webrtc/+/3b9ff38d8a10d46f69680c563...
==========

[commit-bot: I haz the power, 2017-04-27 15:03:47]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/3b9ff38d8a10d46f69680c563...