GN: Tighten up test target visibility + refactorings

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 b57d606a52446369b81945ce9054a844dd2cd3da..647079a38257934bc633187f5a65051214764586 100644
--- a/webrtc/modules/congestion_controller/BUILD.gn
+++ b/webrtc/modules/congestion_controller/BUILD.gn
@@ -62,6 +62,13 @@ rtc_static_library("congestion_controller") {
 if (rtc_include_tests) {
   rtc_source_set("congestion_controller_unittests") {
     testonly = true
+
+    # Skip restricting visibility on mobile platforms since the tests on those
+    # gets additional generated targets which would require many lines here to
+    # cover (which would be confusing to read and hard to maintain).
+    if (!is_android && !is_ios) {
+      visibility = [ "//webrtc/modules:modules_unittests" ]
+    }
     sources = [
       "congestion_controller_unittest.cc",
       "congestion_controller_unittests_helper.cc",
@@ -83,8 +90,8 @@ if (rtc_include_tests) {
       "../../test:test_support",
       "../bitrate_controller:bitrate_controller",
       "../pacing:pacing",
+      "../remote_bitrate_estimator:mock_remote_bitrate_observer",
       "../remote_bitrate_estimator:remote_bitrate_estimator",
-      "../remote_bitrate_estimator:remote_bitrate_estimator_unittests",
       "../rtp_rtcp:rtp_rtcp",
       "//testing/gmock",
     ]