commit db1285676bdcfa9274be9396f34a53e9871ea654
author Danil Chapovalov <danilchap@webrtc.org> Mon Sep 17 13:11:50 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Tue Sep 18 08:08:33 2018 +0000
tree 1de70433dc3acd4dc04ffecd76886c7705841dbf
parent 6d800030ab7efa0c0cb50efee737320d5f1c1d2e

Cleanup modules_common_types

Uninline RTPFragmentaion functions
fix RTPFragmentation move constructor and assign operators (was recursive for win)
replace assert with rtc::dchecked_cast
Remove unused includes and dependencies.
Fix other targets that used those includes transitively instead of directly

Bug: None
Change-Id: I647cb1eda107dc7d87d25234095545bc2842fa40
Reviewed-on: https://webrtc-review.googlesource.com/100500
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24759}

modules/BUILD.gn

diff --git a/modules/BUILD.gn b/modules/BUILD.gn
index e1c1f3a..8127244 100644
--- a/modules/BUILD.gn
+++ b/modules/BUILD.gn
@@ -44,6 +44,7 @@
   visibility = [ "*" ]
   sources = [
     "include/module.h",
+    "include/module_common_types.cc",
     "include/module_common_types.h",
   ]
   deps = [
@@ -51,14 +52,8 @@
     ":module_fec_api",
     "..:webrtc_common",
     "../api:libjingle_peerconnection_api",
-    "../api/transport:network_control",
-    "../api/video:video_frame",
-    "../api/video:video_frame_i420",
     "../modules/rtp_rtcp:rtp_video_header",
-    "../rtc_base:deprecation",
-    "../rtc_base:rtc_base_approved",
-    "video_coding:codec_globals_headers",
-    "//third_party/abseil-cpp/absl/types:optional",
+    "../rtc_base:safe_conversions",
   ]
 }