commit aa3f5da8dc014b7ae426396c944217a1cd25b3ae
author Danil Chapovalov <danilchap@webrtc.org> Thu Nov 14 14:57:33 2019 +0100
committer Commit Bot <commit-bot@chromium.org> Mon Nov 18 15:48:07 2019 +0000
tree 9a04d23a10ad298722bbef853d69aceb25818943
parent acdc22d7845c5dde7c23366110e54e5d26127c85

Fork VCMPacket for PacketBuffer into own struct

it is easier to reduce and eliminate it when it is not bound to legacy video code

Bug: webrtc:10979
Change-Id: I517e298501b3358a914a23ddce40fcb3075d672d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/159707
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#29821}

modules/video_coding/BUILD.gn

diff --git a/modules/video_coding/BUILD.gn b/modules/video_coding/BUILD.gn
index 3c6b522..627000d 100644
--- a/modules/video_coding/BUILD.gn
+++ b/modules/video_coding/BUILD.gn
@@ -44,7 +44,6 @@
   ]
 
   deps = [
-    ":packet",
     "..:module_api",
     "../../api/units:time_delta",
     "../../api/units:timestamp",
@@ -58,24 +57,6 @@
   ]
 }
 
-rtc_library("packet") {
-  visibility = [ "*" ]
-  sources = [
-    "packet.cc",
-    "packet.h",
-  ]
-  deps = [
-    "..:module_api",
-    "../../:webrtc_common",
-    "../../api:rtp_headers",
-    "../../api:rtp_packet_info",
-    "../../api/video:video_frame_type",
-    "../rtp_rtcp:rtp_rtcp_format",
-    "../rtp_rtcp:rtp_video_header",
-    "//third_party/abseil-cpp/absl/types:optional",
-  ]
-}
-
 rtc_library("video_coding") {
   visibility = [ "*" ]
   deps = [
@@ -140,7 +121,6 @@
   deps += [
     ":codec_globals_headers",
     ":encoded_frame",
-    ":packet",
     ":video_codec_interface",
     ":video_coding_utility",
     ":webrtc_vp9_helpers",
@@ -148,6 +128,7 @@
     "..:module_api_public",
     "../../api:fec_controller_api",
     "../../api:rtp_headers",
+    "../../api:rtp_packet_info",
     "../../api/units:data_rate",
     "../../api/units:time_delta",
     "../../api/video:builtin_video_bitrate_allocator_factory",
@@ -215,6 +196,8 @@
     "jitter_buffer.cc",
     "jitter_buffer.h",
     "jitter_buffer_common.h",
+    "packet.cc",
+    "packet.h",
     "receiver.cc",
     "receiver.h",
     "session_info.cc",
@@ -226,14 +209,16 @@
   deps = [
     ":codec_globals_headers",
     ":encoded_frame",
-    ":packet",
     ":video_codec_interface",
     ":video_coding",
     "..:module_api",
     "..:module_api_public",
+    "../../:webrtc_common",
     "../../api:rtp_headers",
+    "../../api:rtp_packet_info",
     "../../api/video:encoded_image",
     "../../api/video:video_frame",
+    "../../api/video:video_frame_type",
     "../../api/video:video_rtp_headers",
     "../../api/video_codecs:video_codecs_api",
     "../../common_video",
@@ -244,6 +229,8 @@
     "../../rtc_base/synchronization:sequence_checker",
     "../../system_wrappers",
     "../../system_wrappers:event_wrapper",
+    "../rtp_rtcp:rtp_rtcp_format",
+    "../rtp_rtcp:rtp_video_header",
     "../utility",
     "//third_party/abseil-cpp/absl/memory",
     "//third_party/abseil-cpp/absl/types:optional",
@@ -874,7 +861,6 @@
       ":codec_globals_headers",
       ":encoded_frame",
       ":nack_module",
-      ":packet",
       ":simulcast_test_fixture_impl",
       ":video_codec_interface",
       ":video_codecs_test_framework",