Set min BWE bitrate form 10kbps to 5kbps and centralize minimum bitrate.

webrtc/modules/congestion_controller/congestion_controller_unittest.cc

 /*
  *  Copyright (c) 2016 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 117 matching lines @@
   controller_->SignalNetworkState(kNetworkDown);
 }
 
 TEST_F(CongestionControllerTest, ResetBweAndBitrates) {
   int new_bitrate = 200000;
   EXPECT_CALL(observer_, OnNetworkChanged(new_bitrate, _, _));
   EXPECT_CALL(*pacer_, SetEstimatedBitrate(new_bitrate));
   controller_->ResetBweAndBitrates(new_bitrate, -1, -1);
 
   // If the bitrate is reset to -1, the new starting bitrate will be
-  // the minimum default bitrate 10000bps.
-  int min_default_bitrate = 10000;
-  EXPECT_CALL(observer_, OnNetworkChanged(min_default_bitrate, _, _));
-  EXPECT_CALL(*pacer_, SetEstimatedBitrate(min_default_bitrate));
+  // the minimum default bitrate kMinBitrateBps.

[minyue-webrtc, 2016/10/13 08:12:31]

does the test depend on the actual value?

[michaelt, 2016/10/13 09:38:41]

Yes the test insert ileal values and expect then that the default values are
passed.

+  EXPECT_CALL(

[the sun, 2016/10/12 15:29:36]

strange formatting - did git cl format do this? 

[michaelt, 2016/10/13 06:34:49]

Yes, this is done by "cl format"

+      observer_,
+      OnNetworkChanged(remote_bitrate_estimator_defines::kMinBitrateBps, _, _));
+  EXPECT_CALL(*pacer_, SetEstimatedBitrate(
+                           remote_bitrate_estimator_defines::kMinBitrateBps));
   controller_->ResetBweAndBitrates(-1, -1, -1);
 }
 
 TEST_F(CongestionControllerTest,
        SignalNetworkStateAndQueueIsFullAndEstimateChange) {
   // Send queue is full
   EXPECT_CALL(*pacer_, ExpectedQueueTimeMs())
       .WillRepeatedly(Return(PacedSender::kMaxQueueLengthMs + 1));
   EXPECT_CALL(observer_, OnNetworkChanged(0, _, _));
   controller_->Process();
@@ skipping 30 matching lines @@
   controller_->SignalNetworkState(kNetworkDown);
   EXPECT_EQ(0, controller_->GetPacerQueuingDelayMs());
 
   // Network is up, pacer delay should be reported.
   controller_->SignalNetworkState(kNetworkUp);
   EXPECT_EQ(kQueueTimeMs, controller_->GetPacerQueuingDelayMs());
 }
 
 }  // namespace test
 }  // namespace webrtc