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

webrtc/modules/congestion_controller/include/congestion_controller.h

Index: webrtc/modules/congestion_controller/include/congestion_controller.h
diff --git a/webrtc/modules/congestion_controller/include/congestion_controller.h b/webrtc/modules/congestion_controller/include/congestion_controller.h
index eb6db3371226352bc921889226084c985b6b3d05..c734a731476d71317db0f80adeaac877e376bc7d 100644
--- a/webrtc/modules/congestion_controller/include/congestion_controller.h
+++ b/webrtc/modules/congestion_controller/include/congestion_controller.h
@@ -54,6 +54,9 @@ class CongestionController : public CallStatsObserver, public Module {
    protected:
     virtual ~Observer() {}
   };
+
+  static int GetMinBitrateBps();

[stefan-webrtc, 2016/11/01 09:09:15]

I find it slightly weird that we expose this in the congestion controller
interface, as it's basically only used within the congestion controller
implementation. As I see it, CongestionController is implemented using the
classes in modules/bitrate_controller, modules/remote_bitrate_estimator,
modules/pacing, etc. I think we may end up moving all of those into
modules/congestion_controller down the line.

There may not be a better place for it right now though... Any suggestions?
Otherwise we'll go ahead with this.

[michaelt, 2016/11/01 09:21:55]

Moving the CongestionController related classes in to
modules/congestion_controller sounds like a good idea.

At the moment I see no better place to define GetMinBitrateBps. We may pass in
future the value down, or remove it in some places. So that we don't have to
define it globally. 

+
   CongestionController(Clock* clock,
                        Observer* observer,
                        RemoteBitrateObserver* remote_bitrate_observer,