Refactor webrtc/{api,audio} and modules/audio_coding for GN check
This moves some GN check configurations out of .gn to individual targets.
The now checked targets are:
"//webrtc/api/*",
"//webrtc/audio/*",
"//webrtc/modules/audio_coding/*",
Many targets were fixed by adding dependencies, but the ones that
requires more refactorings are left with the check_includes attribute
set to false instead.
Make //webrtc/test:test_support a public dep of //webrtc/test:test_main
to avoid having to add that to all users of it.
BUG=webrtc:6828
NOTRY=True
Review-Url: https://codereview.webrtc.org/2556943003
Cr-Commit-Position: refs/heads/master@{#15461}
diff --git a/webrtc/audio/BUILD.gn b/webrtc/audio/BUILD.gn
index bfc2372..0669c60 100644
--- a/webrtc/audio/BUILD.gn
+++ b/webrtc/audio/BUILD.gn
@@ -30,11 +30,17 @@
deps = [
"..:webrtc_common",
"../api:audio_mixer_api",
+ "../api:call_api",
"../base:rtc_base_approved",
+ "../base:rtc_task_queue",
"../call:call_interfaces",
"../common_audio",
"../modules/audio_device",
"../modules/audio_processing",
+ "../modules/congestion_controller:congestion_controller",
+ "../modules/pacing:pacing",
+ "../modules/remote_bitrate_estimator:remote_bitrate_estimator",
+ "../modules/rtp_rtcp:rtp_rtcp",
"../system_wrappers",
"../voice_engine",
]
@@ -42,6 +48,11 @@
if (rtc_include_tests) {
rtc_source_set("audio_tests") {
testonly = true
+
+ # TODO(kjellander): Remove (bugs.webrtc.org/6828)
+ # This needs remote_bitrate_estimator to be moved to webrtc/api first.
+ check_includes = false
+
sources = [
"audio_receive_stream_unittest.cc",
"audio_send_stream_unittest.cc",
@@ -52,8 +63,11 @@
":audio",
"../api:mock_audio_mixer",
"../base:rtc_base_approved",
+ "../base:rtc_task_queue",
"../modules/audio_device:mock_audio_device",
"../modules/audio_mixer:audio_mixer_impl",
+ "../modules/congestion_controller:congestion_controller",
+ "../modules/pacing:pacing",
"../test:test_common",
"../test:test_support",
"utility:audio_frame_operations",
