Reland "WebRtcVoiceMediaChannel::AddRecvStream: Don't call SetRecPayloadType"

Reland "WebRtcVoiceMediaChannel::AddRecvStream: Don't call SetRecPayloadType"

BUG=webrtc:5805
Review-Url: https://codereview.webrtc.org/2774833003
Cr-Commit-Position: refs/heads/master@{#17391}
Committed: https://chromium.googlesource.com/external/webrtc/+/1c07c70d88eca07888bf242921dc8dfc3be95737

[kwiberg-webrtc, 2017-03-27 12:43:57]

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

[commit-bot: I haz the power, 2017-03-27 12:44:02]

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

[kwiberg-webrtc, 2017-03-27 12:50:10]

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

[commit-bot: I haz the power, 2017-03-27 13:26:45]

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

[commit-bot: I haz the power, 2017-03-27 13:26:46]

Dry run: This issue passed the CQ dry run.

[the sun, 2017-03-27 14:09:58]

lgtm

[the sun, 2017-03-27 14:13:09]

The CQ bit was checked by solenberg@webrtc.org

[commit-bot: I haz the power, 2017-03-27 14:13:15]

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-03-27 14:15:50]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490623989553110,
"parent_rev": "b5eae743414d62555c679f662d119ec303b9a857", "commit_rev":
"1c07c70d88eca07888bf242921dc8dfc3be95737"}

[commit-bot: I haz the power, 2017-03-27 14:15:54]

Description was changed from

==========
Reland "WebRtcVoiceMediaChannel::AddRecvStream: Don't call SetRecPayloadType"

BUG=webrtc:5805
==========

to

==========
Reland "WebRtcVoiceMediaChannel::AddRecvStream: Don't call SetRecPayloadType"

BUG=webrtc:5805

Review-Url: https://codereview.webrtc.org/2774833003
Cr-Commit-Position: refs/heads/master@{#17391}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1c07c70d88eca07888bf24292...
==========

[commit-bot: I haz the power, 2017-03-27 14:15:55]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/1c07c70d88eca07888bf24292...

[oprypin_webrtc, 2017-03-28 18:27:26]

Description was changed from

==========
Reland "WebRtcVoiceMediaChannel::AddRecvStream: Don't call SetRecPayloadType"

BUG=webrtc:5805

Review-Url: https://codereview.webrtc.org/2774833003
Cr-Commit-Position: refs/heads/master@{#17391}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1c07c70d88eca07888bf24292...
==========

to

==========
Reland "WebRtcVoiceMediaChannel::AddRecvStream: Don't call SetRecPayloadType"

BUG=webrtc:5805

Review-Url: https://codereview.webrtc.org/2774833003
Cr-Commit-Position: refs/heads/master@{#17391}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1c07c70d88eca07888bf24292...
==========

[oprypin_webrtc, 2017-03-28 18:32:34]

The reland still causes the breakage of the test on the perfbot - see
https://build.chromium.org/p/client.webrtc.perf/builders/Linux%20Trusty/build...

The test in question:
https://chromium.googlesource.com/external/webrtc/+/master/webrtc/audio/test/...

This is reproducible locally. Seems like it's getting no audio on the receive
side. The error you can see on the perfbot is just a complaint that the audio
file is empty (which is indeed shorter than 1/4 sec).

I guess this indicates that this type of failure could be handled better in the
test (maybe even on trybots). But we still need to get to the bottom of the
problem and fix the test.