Fix issue with conflicting behavior if setting a max BW with b=AS on both audio and video.

webrtc/media/engine/webrtcvoiceengine.cc

Index: webrtc/media/engine/webrtcvoiceengine.cc
diff --git a/webrtc/media/engine/webrtcvoiceengine.cc b/webrtc/media/engine/webrtcvoiceengine.cc
index 7c6d7940f685cd9e11aa4f90227b93582ccf6ce0..2db7061364088b859faa2b10df0ccf6265c15420 100644
--- a/webrtc/media/engine/webrtcvoiceengine.cc
+++ b/webrtc/media/engine/webrtcvoiceengine.cc
@@ -1638,12 +1638,6 @@ bool WebRtcVoiceMediaChannel::SetSendParameters(
     return false;
   }
 
-  if (params.max_bandwidth_bps >= 0) {
-    // Note that max_bandwidth_bps intentionally takes priority over the
-    // bitrate config for the codec.
-    bitrate_config_.max_bitrate_bps =
-        params.max_bandwidth_bps == 0 ? -1 : params.max_bandwidth_bps;
-  }
   call_->SetBitrateConfig(bitrate_config_);

[minyue-webrtc, 2017/03/27 09:22:06]

I think there is no need to call this if we remove resetting of bitrate_config_

we may move this to  SetSendCodecs(), and the need of bitrate_config_ as member
var goes away.

[stefan-webrtc, 2017/03/27 10:10:27]

Done.

 
   if (!ValidateRtpExtensions(params.extensions)) {