commit fd6c744a7cacc2c9699c1e473781be45c1df3aed
author Harald Alvestrand <hta@webrtc.org> Tue Feb 08 07:52:10 2022
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Feb 08 09:26:24 2022
tree 911c243b167a1f26d44cfd2e210ad527b03d18da
parent 3aa9937e48fec17bfdcc3a65c141efb527a9d67c

Break out rtc_stats_collector and rtc_stats_traversal

Bug: webrtc:13634
Change-Id: I94f8cfe5511ff0798ec4a5ad7df28f3e7b98165d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251241
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35945}

pc/BUILD.gn

diff --git a/pc/BUILD.gn b/pc/BUILD.gn
index eb119c5..c24e8369 100644
--- a/pc/BUILD.gn
+++ b/pc/BUILD.gn
@@ -244,10 +244,6 @@
     "peer_connection.h",
     "peer_connection_factory.cc",
     "peer_connection_factory.h",
-    "rtc_stats_collector.cc",
-    "rtc_stats_collector.h",
-    "rtc_stats_traversal.cc",
-    "rtc_stats_traversal.h",
     "sdp_offer_answer.cc",  # TODO: Make separate target when not circular
     "sdp_offer_answer.h",  # dependent on peerconnection.h
     "stats_collector.cc",
@@ -488,7 +484,66 @@
 }
 rtc_source_set("rtc_stats_collector") {
   visibility = [ ":*" ]
+  sources = [
+    "rtc_stats_collector.cc",
+    "rtc_stats_collector.h",
+  ]
+  deps = [
+    ":data_channel_utils",
+    ":peer_connection_internal",
+    ":rtc_pc_base",
+    ":rtc_stats_traversal",
+    ":rtp_receiver",
+    ":rtp_sender",
+    ":rtp_transceiver",
+    ":sctp_data_channel",
+    ":track_media_info_map",
+    ":webrtc_sdp",
+    "../api:array_view",
+    "../api:libjingle_peerconnection_api",
+    "../api:media_stream_interface",
+    "../api:rtc_stats_api",
+    "../api:rtp_parameters",
+    "../api:scoped_refptr",
+    "../api:sequence_checker",
+    "../api/task_queue:task_queue",
+    "../api/video:video_rtp_headers",
+    "../call:call_interfaces",
+    "../common_video:common_video",
+    "../media:rtc_media_base",
+    "../modules/audio_processing:audio_processing_statistics",
+    "../modules/rtp_rtcp:rtp_rtcp_format",
+    "../p2p:rtc_p2p",
+    "../rtc_base:checks",
+    "../rtc_base:ip_address",
+    "../rtc_base:logging",
+    "../rtc_base:network_constants",
+    "../rtc_base:refcount",
+    "../rtc_base:rtc_base",
+    "../rtc_base:rtc_base_approved",
+    "../rtc_base:rtc_event",
+    "../rtc_base:socket_address",
+    "../rtc_base:stringutils",
+    "../rtc_base:threading",
+    "../rtc_base:timeutils",
+    "../rtc_base/third_party/sigslot:sigslot",
+  ]
+  absl_deps = [ "//third_party/abseil-cpp/absl/types:optional" ]
 }
+
+rtc_source_set("rtc_stats_traversal") {
+  visibility = [ ":*" ]
+  sources = [
+    "rtc_stats_traversal.cc",
+    "rtc_stats_traversal.h",
+  ]
+  deps = [
+    "../api:rtc_stats_api",
+    "../api:scoped_refptr",
+    "../rtc_base:checks",
+  ]
+}
+
 rtc_source_set("sdp_offer_answer") {
   visibility = [ ":*" ]
 }
@@ -514,9 +569,7 @@
 rtc_source_set("peer_connection") {
   visibility = [ ":*" ]
 }
-rtc_source_set("rtc_stats_traversal") {
-  visibility = [ ":*" ]
-}
+
 rtc_source_set("sdp_serializer") {
   visibility = [ ":*" ]
   sources = [
@@ -1358,6 +1411,8 @@
       ":proxy",
       ":remote_audio_source",
       ":rtc_pc_base",
+      ":rtc_stats_collector",
+      ":rtc_stats_traversal",
       ":rtp_parameters_conversion",
       ":rtp_receiver",
       ":rtp_sender",