GN: Tighten up test target visibility + refactorings

webrtc/modules/rtp_rtcp/BUILD.gn

Index: webrtc/modules/rtp_rtcp/BUILD.gn
diff --git a/webrtc/modules/rtp_rtcp/BUILD.gn b/webrtc/modules/rtp_rtcp/BUILD.gn
index ec1766641ed9f94727870d720e378b27669b3f94..ecbd6ed21c214763f341e1c3cde875d8a462a143 100644
--- a/webrtc/modules/rtp_rtcp/BUILD.gn
+++ b/webrtc/modules/rtp_rtcp/BUILD.gn
@@ -210,6 +210,9 @@ if (rtc_include_tests) {
 
   rtc_source_set("rtp_rtcp_modules_tests") {
     testonly = true
+    if (!is_android && !is_ios) {  # Generated targets makes this check break.
+      visibility = [ "//webrtc/modules:modules_tests" ]
+    }
     sources = [
       "test/testFec/test_fec.cc",
     ]
@@ -224,13 +227,44 @@ if (rtc_include_tests) {
     }
   }
 
-  rtc_source_set("rtp_rtcp_unittests") {
+  rtc_source_set("mock_rtp_rtcp") {
     testonly = true
     sources = [
       "mocks/mock_rtp_rtcp.h",
-      "source/byte_io_unittest.cc",
+    ]
+    deps = [
+      ":rtp_rtcp",
+      "../../base:rtc_base_approved",
+      "../../test:test_support",
+    ]
+  }
+
+  rtc_source_set("fec_test_helper") {
+    testonly = true
+    sources = [
       "source/fec_test_helper.cc",
       "source/fec_test_helper.h",
+    ]
+    deps = [
+      ":rtp_rtcp",
+      "../../base:rtc_base_approved",
+    ]
+
+    # TODO(jschuh): bugs.webrtc.org/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" ]
+    }
+  }
+
+  rtc_source_set("rtp_rtcp_unittests") {
+    testonly = true
+    if (!is_android && !is_ios) {  # Generated targets makes this check break.
+      visibility = [ "//webrtc/modules:modules_unittests" ]
+    }
+    sources = [
+      "source/byte_io_unittest.cc",
       "source/flexfec_header_reader_writer_unittest.cc",
       "source/flexfec_receiver_unittest.cc",
       "source/flexfec_sender_unittest.cc",
@@ -291,6 +325,8 @@ if (rtc_include_tests) {
       "test/testAPI/test_api_video.cc",
     ]
     deps = [
+      ":fec_test_helper",
+      ":mock_rtp_rtcp",
       ":rtp_rtcp",
       "../..:webrtc_common",
       "../../api:transport_api",