New targets call:rtp_interfaces, call:rtp_receiver, call:rtp_sender.

webrtc/call/BUILD.gn

Index: webrtc/call/BUILD.gn
diff --git a/webrtc/call/BUILD.gn b/webrtc/call/BUILD.gn
index a6afe302409949a2a6609bba13e1232ba320bfe9..2f01b8dac11ec98ec4aa33e5d147e2689f648444 100644
--- a/webrtc/call/BUILD.gn
+++ b/webrtc/call/BUILD.gn
@@ -16,12 +16,11 @@ rtc_source_set("call_interfaces") {
     "audio_state.h",
     "call.h",
     "flexfec_receive_stream.h",
-    "rtp_demuxer.h",
-    "rtp_transport_controller_send_interface.h",
     "syncable.cc",
     "syncable.h",
   ]
   deps = [
+    ":rtp_interfaces",
     "..:video_stream_api",
     "..:webrtc_common",
     "../api:audio_mixer_api",
@@ -33,17 +32,47 @@ rtc_source_set("call_interfaces") {
   ]
 }
 
+# TODO(nisse): These RTP targets should be moved elsewhere
+# when interfaces have stabilized.
+rtc_source_set("rtp_interfaces") {
+  sources = [
+    "rtp_demuxer.h",
+    "rtp_transport_controller_send_interface.h",
+  ]
+}
+
+rtc_source_set("rtp_receiver") {
+  sources = [
+    "rtp_demuxer.cc",

[the sun, 2017/05/31 13:06:29]

It seems rtp_demuxer should be split so that the clients only needing the
interface (RtpPacketSinkinterface?) would only pull that in. The .cc and .h
shouldn't live in separate targets.

[nisse-webrtc, 2017/05/31 13:16:50]

Ok, so I'd split out the RtpPacketSinkInterface to it's own header file, and put
that in the rtp_interface target?

+    "rtp_demuxer.h",
+    "rtx_receive_stream.cc",
+    "rtx_receive_stream.h",
+  ]
+  deps = [
+    ":rtp_interfaces",

[danilchap, 2017/05/31 11:56:59]

is this dependency needed?

[nisse-webrtc, 2017/05/31 12:01:09]

It looks like rtp_demuxer.h was accidentally listed in two targets. If it's only
in rtp_interfaces, this dependency is required.

Question is, should we have the rtp_interfaces target or not?

+    "../base:rtc_base_approved",
+    "../modules/rtp_rtcp",
+  ]
+}
+
+rtc_source_set("rtp_sender") {
+  sources = [
+    "rtp_transport_controller_send.cc",
+    "rtp_transport_controller_send.h",
+  ]
+  deps = [
+    ":rtp_interfaces",
+    "../base:rtc_base_approved",
+    "../modules/congestion_controller",
+  ]
+}
+
 rtc_static_library("call") {
   sources = [
     "bitrate_allocator.cc",
     "call.cc",
     "flexfec_receive_stream_impl.cc",
     "flexfec_receive_stream_impl.h",
-    "rtp_demuxer.cc",
-    "rtp_transport_controller_send.cc",
-    "rtp_transport_controller_send.h",
-    "rtx_receive_stream.cc",
-    "rtx_receive_stream.h",
   ]
 
   if (!build_with_chromium && is_clang) {
@@ -58,6 +87,9 @@ rtc_static_library("call") {
 
   deps = [
     ":call_interfaces",
+    ":rtp_interfaces",
+    ":rtp_receiver",
+    ":rtp_sender",
     "..:webrtc_common",
     "../api:transport_api",
     "../audio",
@@ -93,6 +125,9 @@ if (rtc_include_tests) {
     ]
     deps = [
       ":call",
+      ":rtp_interfaces",
+      ":rtp_receiver",
+      ":rtp_sender",
       "../api:mock_audio_mixer",
       "../base:rtc_base_approved",
       "../logging:rtc_event_log_api",