Refactor bitrate_controller, remote_bitrate_estimator and congestion_contoller for gn check

webrtc/modules/congestion_controller/BUILD.gn

Index: webrtc/modules/congestion_controller/BUILD.gn
diff --git a/webrtc/modules/congestion_controller/BUILD.gn b/webrtc/modules/congestion_controller/BUILD.gn
index d2dc680adbb963906f380f64c23b20611d00f7ab..7b2d061078a83ed465b1a1c3240a34869b8a0d8c 100644
--- a/webrtc/modules/congestion_controller/BUILD.gn
+++ b/webrtc/modules/congestion_controller/BUILD.gn
@@ -43,8 +43,15 @@ rtc_static_library("congestion_controller") {
   }
 
   deps = [
+    "../..:webrtc_common",
     "../../base:rtc_analytics",
+    "../../base:rtc_base",
+    "../../base:rtc_base_approved",
+    "../../system_wrappers:system_wrappers",

[kjellander_webrtc, 2016/12/27 06:22:54]

GN picks a target named similar to the directory if omitted, so
"../../system_wrappers", is enough here. I don't think the style guide says it's
mandatory, but I think it improves the readability.

[mbonadei, 2016/12/27 14:51:17]

Acknowledged.

     "../bitrate_controller",
     "../pacing",
+    "../remote_bitrate_estimator:remote_bitrate_estimator",

[kjellander_webrtc, 2016/12/27 06:22:54]

same as above; please leave out the implicit target names for increased
readability.

[mbonadei, 2016/12/27 14:51:17]

Acknowledged.

+    "../rtp_rtcp:rtp_rtcp",
+    "../utility:utility",
   ]
 }