Enabling 'gn check' on webrtc/video

webrtc/video/BUILD.gn

Index: webrtc/video/BUILD.gn
diff --git a/webrtc/video/BUILD.gn b/webrtc/video/BUILD.gn
index 524f84b3688a36e1326839052dc15ae0f7319fdc..52afc93064771e1878502ec9fd4060dbd9899ef1 100644
--- a/webrtc/video/BUILD.gn
+++ b/webrtc/video/BUILD.gn
@@ -60,6 +60,7 @@ rtc_static_library("video") {
     "../base:rtc_base_approved",
     "../base:rtc_numerics",
     "../base:rtc_task_queue",
+    "../call:call_interfaces",
     "../common_video",
     "../logging:rtc_event_log_api",
     "../media:rtc_media_base",
@@ -70,6 +71,8 @@ rtc_static_library("video") {
     "../modules/rtp_rtcp",
     "../modules/utility",
     "../modules/video_coding",
+    "../modules/video_coding:video_coding_utility",
+    "../modules/video_coding:webrtc_vp8",
     "../modules/video_processing",
     "../system_wrappers",
     "../voice_engine",
@@ -86,8 +89,19 @@ if (rtc_include_tests) {
     deps = [
       "../base:rtc_base_tests_utils",
       "../base:rtc_task_queue",
+      "../call:call_interfaces",
+      "../common_video",
+      "../logging:rtc_event_log_api",
       "../media:rtc_media_base",
+      "../modules/audio_mixer:audio_mixer_impl",
+      "../modules/rtp_rtcp",
+      "../modules/video_coding:webrtc_h264",
+      "../modules/video_coding:webrtc_vp8",
+      "../modules/video_coding:webrtc_vp9",
       "../system_wrappers",
+      "../test:test_common",
+      "../test:test_support",
+      "../voice_engine",
       "//testing/gtest",
       "//webrtc/test:test_renderer",
       "//webrtc/test:video_test_common",
@@ -105,6 +119,8 @@ if (rtc_include_tests) {
     ]
     deps = [
       ":video_quality_test",
+      "../test:field_trial",
+      "../test:test_support",
       "//testing/gtest",
       "//webrtc/test:test_common",
     ]
@@ -129,6 +145,7 @@ if (rtc_include_tests) {
       "../test:run_test",
       "../test:test_common",
       "../test:test_renderer",
+      "../test:test_support",
       "//testing/gmock",
       "//testing/gtest",
       "//third_party/gflags",
@@ -147,11 +164,13 @@ if (rtc_include_tests) {
 
     deps = [
       ":video_quality_test",
+      "../base:rtc_base_approved",
       "../system_wrappers:metrics_default",
       "../test:field_trial",
       "../test:run_test",
       "../test:test_common",
       "../test:test_renderer",
+      "../test:test_support",
       "//third_party/gflags",
     ]
     if (!build_with_chromium && is_clang) {
@@ -167,12 +186,22 @@ if (rtc_include_tests) {
       "replay.cc",
     ]
     deps = [
+      "..:webrtc_common",
       "../api/video_codecs:video_codecs_api",
+      "../base:rtc_base_approved",
+      "../call:call_interfaces",
+      "../common_video",
+      "../logging:rtc_event_log_api",
+      "../modules/rtp_rtcp",
+      "../system_wrappers",
       "../system_wrappers:metrics_default",
       "../test:field_trial",
+      "../test:rtp_test_utils",
       "../test:run_test",
       "../test:test_common",
       "../test:test_renderer",
+      "../test:test_support",
+      "../test:video_test_common",
       "//third_party/gflags",
     ]
     if (!build_with_chromium && is_clang) {
@@ -205,7 +234,34 @@ if (rtc_include_tests) {
     ]
     deps = [
       ":video",
+      "../api:video_frame_api",
+      "../api/video_codecs:video_codecs_api",
+      "../base:rtc_base_approved",
+      "../base:rtc_base_tests_utils",
+      "../call:call_interfaces",
+      "../common_video",
+      "../logging:rtc_event_log_api",
       "../media:rtc_media_base",
+      "../media:rtc_media_tests_utils",
+      "../modules/pacing",
+      "../modules/rtp_rtcp",
+      "../modules/rtp_rtcp:rtp_rtcp_unittests",

[kjellander_webrtc, 2017/04/25 08:50:13]

Ah, I missed that this target now pulled in additional tests with this
dependency.

Luckily I'm fixing that and preventing such from happening in the future with my
https://codereview.webrtc.org/2828793003/

[mbonadei, 2017/04/25 09:08:38]

Thanks, I missed that too!

+      "../modules/utility",
+      "../modules/video_coding",
+      "../modules/video_coding:video_coding_utility",
+      "../modules/video_coding:webrtc_h264",
+      "../modules/video_coding:webrtc_vp8",
+      "../modules/video_coding:webrtc_vp9",
+      "../system_wrappers",
+      "../system_wrappers:field_trial_default",
+      "../system_wrappers:metrics_api",
+      "../system_wrappers:metrics_default",
+      "../test:direct_transport",
+      "../test:field_trial",
+      "../test:rtp_test_utils",
+      "../test:test_common",
+      "../test:test_support",
+      "../test:video_test_common",
       "//testing/gmock",
       "//testing/gtest",
     ]