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

webrtc/modules/congestion_controller/congestion_controller_unittest.cc

Index: webrtc/modules/congestion_controller/congestion_controller_unittest.cc
diff --git a/webrtc/modules/congestion_controller/congestion_controller_unittest.cc b/webrtc/modules/congestion_controller/congestion_controller_unittest.cc
index e840294d9f09a96da12b5631606ab717c2227e16..5fd3b52aa38701a759a2d26b7741ace3dbfa44bd 100644
--- a/webrtc/modules/congestion_controller/congestion_controller_unittest.cc
+++ b/webrtc/modules/congestion_controller/congestion_controller_unittest.cc
@@ -135,10 +135,12 @@ TEST_F(CongestionControllerTest, ResetBweAndBitrates) {
   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);
 }