commit 99175c6eb3f074630ad7695addb2785ec38fc0eb
author Patrik Höglund <phoglund@webrtc.org> Mon Jan 08 11:05:10 2018 +0100
committer Commit Bot <commit-bot@chromium.org> Mon Jan 08 11:51:52 2018 +0000
tree ea073dc5607d6afb95c8f4b014538d9619668b3d
parent c492bf1958875af2f991ebd575300165accbd008

Add untracked headers to video_coding.

This creates a new target for pure defines and interfaces. I think
that makes sense (though include/ makes it harder to see when .cc and
.h files should live together).

Bug: webrtc:7620
Change-Id: Ifb0f50faf99166202836c0446feed3443eb52c6e
Reviewed-on: https://webrtc-review.googlesource.com/34657
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#21516}

modules/video_coding/BUILD.gn

diff --git a/modules/video_coding/BUILD.gn b/modules/video_coding/BUILD.gn
index fa8bc55..2ca7f14 100644
--- a/modules/video_coding/BUILD.gn
+++ b/modules/video_coding/BUILD.gn
@@ -36,8 +36,11 @@
     "histogram.cc",
     "histogram.h",
     "include/video_codec_initializer.h",
+
+    # TODO(bugs.webrtc.org/7620: remove these when downstream is updated).
+    "include/video_codec_interface.h",
     "include/video_coding.h",
-    "include/video_coding_defines.h",
+    "include/video_error_codes.h",
     "inter_frame_delay.cc",
     "inter_frame_delay.h",
     "internal_defines.h",
@@ -90,6 +93,7 @@
 
   deps = [
     ":codec_globals_headers",
+    ":video_codec_interface",
     ":video_coding_utility",
     ":webrtc_h264",
     ":webrtc_i420",
@@ -104,6 +108,7 @@
     "../../api:optional",
     "../../api:video_frame_api",
     "../../api:video_frame_api_i420",
+    "../../api/video_codecs:video_codecs_api",
     "../../call:video_stream_api",
     "../../common_video",
     "../../rtc_base:checks",
@@ -121,6 +126,22 @@
   ]
 }
 
+rtc_source_set("video_codec_interface") {
+  sources = [
+    "include/video_codec_interface.h",
+    "include/video_coding_defines.h",
+    "include/video_error_codes.h",
+  ]
+  deps = [
+    "..:module_api",
+    "../..:typedefs",
+    "../..:webrtc_common",
+    "../../api:video_frame_api",
+    "../../api/video_codecs:video_codecs_api",
+    "../../common_video:common_video",
+  ]
+}
+
 rtc_source_set("mock_headers") {
   testonly = true
   sources = [
@@ -128,6 +149,7 @@
     "include/mock/mock_video_codec_interface.h",
   ]
   deps = [
+    ":video_codec_interface",
     ":video_coding_utility",
     "../..:webrtc_common",
     "../../:typedefs",
@@ -171,6 +193,7 @@
   }
 
   deps = [
+    ":video_codec_interface",
     "..:module_api",
     "../..:webrtc_common",
     "../../:typedefs",
@@ -201,6 +224,7 @@
 
   defines = []
   deps = [
+    ":video_codec_interface",
     ":video_coding_utility",
     "../../api:video_frame_api_i420",
     "../../api/video_codecs:video_codecs_api",
@@ -242,6 +266,7 @@
   }
 
   deps = [
+    ":video_codec_interface",
     ":video_coding_utility",
     "../..:webrtc_common",
     "../../:typedefs",
@@ -268,6 +293,7 @@
   }
 
   deps = [
+    ":video_codec_interface",
     ":video_coding_utility",
     "..:module_api",
     "../..:webrtc_common",
@@ -327,6 +353,7 @@
   }
 
   deps = [
+    ":video_codec_interface",
     ":video_coding_utility",
     ":webrtc_vp8_helpers",
     "..:module_api",
@@ -371,6 +398,7 @@
   }
 
   deps = [
+    ":video_codec_interface",
     ":video_coding_utility",
     "..:module_api",
     "../../common_video",
@@ -423,6 +451,7 @@
 
     deps = [
       ":mock_headers",
+      ":video_codec_interface",
       ":video_coding",
       ":webrtc_vp8",
       ":webrtc_vp8_helpers",
@@ -458,6 +487,7 @@
 
     deps = [
       ":codec_globals_headers",
+      ":video_codec_interface",
       ":video_coding",
       ":video_coding_utility",
       ":webrtc_vp8",
@@ -512,6 +542,7 @@
     ]
 
     deps = [
+      ":video_codec_interface",
       ":video_codecs_test_framework",
       ":video_coding",
       ":video_coding_utility",
@@ -627,6 +658,7 @@
     deps = [
       ":mock_headers",
       ":simulcast_test_utility",
+      ":video_codec_interface",
       ":video_codecs_test_framework",
       ":video_coding",
       ":video_coding_utility",