Refactor bitrate_controller, remote_bitrate_estimator and congestion_contoller for gn check

webrtc/modules/congestion_controller/BUILD.gn

 # Copyright (c) 2014 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.
 
 import("../../build/webrtc.gni")
 
@@ skipping 25 matching lines @@
 
   # TODO(jschuh): Bug 1348: fix this warning.
   configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
 
   if (!build_with_chromium && is_clang) {
     # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
     suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
   }
 
   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",
   ]
 }