Control combined_audio_video_bwe with config bool.

Control combined_audio_video_bwe with config bool.

Permits setting RTP extensions for AudioReceiveStream without enabling
combined A/V BWE. This prevents spamming the log with "Failed to find
extension id:".

BUG=webrtc:4870
R=mflodman@webrtc.org, stefan@webrtc.org

Committed: https://crrev.com/6bb1b6e7fe5631e9f218b80292df5b64623c5616
Cr-Commit-Position: refs/heads/master@{#9633}

[pbos-webrtc, 2015-07-24 09:44:23]

PTAL

[pbos-webrtc, 2015-07-24 09:44:30]

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

[commit-bot: I haz the power, 2015-07-24 09:44:38]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1256803004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1256803004/1

[stefan-webrtc, 2015-07-24 09:57:47]

lgtm

https://codereview.webrtc.org/1256803004/diff/1/talk/media/webrtc/webrtcvoice...
File talk/media/webrtc/webrtcvoiceengine_unittest.cc (right):

https://codereview.webrtc.org/1256803004/diff/1/talk/media/webrtc/webrtcvoice...
talk/media/webrtc/webrtcvoiceengine_unittest.cc:3437:
TEST_F(WebRtcVoiceEngineTestFake, SetCall) {
Rename

[pbos-webrtc, 2015-07-24 09:59:13]

rename -> SetCallConfiguresAudioReceiveChannels

[pbos-webrtc, 2015-07-24 11:15:25]

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

[commit-bot: I haz the power, 2015-07-24 11:15:32]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1256803004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1256803004/20001

[mflodman, 2015-07-24 12:28:23]

One comment, then LGTM.

https://codereview.webrtc.org/1256803004/diff/20001/webrtc/audio_receive_stre...
File webrtc/audio_receive_stream.h (right):

https://codereview.webrtc.org/1256803004/diff/20001/webrtc/audio_receive_stre...
webrtc/audio_receive_stream.h:64: bool combined_audio_video_bwe = true;
I'd prefer to set this to false until we have verified combined a/v BWE.

[stefan-webrtc, 2015-07-24 12:35:10]

https://codereview.webrtc.org/1256803004/diff/20001/webrtc/audio_receive_stre...
File webrtc/audio_receive_stream.h (right):

https://codereview.webrtc.org/1256803004/diff/20001/webrtc/audio_receive_stre...
webrtc/audio_receive_stream.h:64: bool combined_audio_video_bwe = true;
On 2015/07/24 12:28:22, mflodman_OOO wrote:
> I'd prefer to set this to false until we have verified combined a/v BWE.

Good point, I agree.

[commit-bot: I haz the power, 2015-07-24 12:37:37]

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

[commit-bot: I haz the power, 2015-07-24 12:37:38]

Dry run: This issue passed the CQ dry run.

[pbos-webrtc, 2015-07-24 12:39:54]

default config combined_audio_video_bwe to false

[pbos-webrtc, 2015-07-24 12:40:02]

The CQ bit was checked by pbos@webrtc.org

[pbos-webrtc, 2015-07-24 12:40:02]

The patchset sent to the CQ was uploaded after l-g-t-m from stefan@webrtc.org,
mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1256803004/#ps30001 (title:
"default config combined_audio_video_bwe to false")

[commit-bot: I haz the power, 2015-07-24 12:40:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1256803004/30001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1256803004/30001

[commit-bot: I haz the power, 2015-07-24 14:10:23]

Committed patchset #3 (id:30001)

[commit-bot: I haz the power, 2015-07-24 14:10:36]

Patchset 3 (id:??) landed as
https://crrev.com/6bb1b6e7fe5631e9f218b80292df5b64623c5616
Cr-Commit-Position: refs/heads/master@{#9633}