Pass time constant to bwe smoothing filter.

webrtc/voice_engine/voe_codec_impl.cc

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 120 matching lines @@
 }
 
 int VoECodecImpl::SetBitRate(int channel, int bitrate_bps) {
   WEBRTC_TRACE(kTraceApiCall, kTraceVoice, VoEId(_shared->instance_id(), -1),
                "SetBitRate(bitrate_bps=%d)", bitrate_bps);
   if (!_shared->statistics().Initialized()) {
     _shared->SetLastError(VE_NOT_INITED, kTraceError);
     return -1;
   }
   _shared->channel_manager().GetChannel(channel).channel()->SetBitRate(
-      bitrate_bps);
+      bitrate_bps, 0);

[minyue-webrtc, 2016/11/28 12:00:00]

Should we give something more meaningful? "0" seems unexpected.

[minyue-webrtc, 2016/11/28 13:14:42]

?

[michaelt, 2016/11/28 13:30:09]

Oh sorry missed that one. This function is not used at all. I think we can
remove the function. But we should do it in a different CL.

I could use 
constexpr int64_t kDefaultProbingIntervalMs = 3000;
To make this a bit more clear here.

   return 0;
 }
 
 int VoECodecImpl::GetRecCodec(int channel, CodecInst& codec) {
   if (!_shared->statistics().Initialized()) {
     _shared->SetLastError(VE_NOT_INITED, kTraceError);
     return -1;
   }
   voe::ChannelOwner ch = _shared->channel_manager().GetChannel(channel);
   voe::Channel* channelPtr = ch.channel();
@@ skipping 237 matching lines @@
     _shared->SetLastError(VE_CHANNEL_NOT_VALID, kTraceError,
                           "GetOpusDtx failed to locate channel");
     return -1;
   }
   return channelPtr->GetOpusDtx(enabled);
 }
 
 #endif  // WEBRTC_VOICE_ENGINE_CODEC_API
 
 }  // namespace webrtc