Move video send/receive stream headers to webrtc/call.
Moved the headers video_receive_stream.h and video_send_stream.h from
webrtc/ into webrtc/call/ as part of the Slim and Modular work.
The GN target webrtc:video_stream_api has moved to
webrtc/call:video_stream_api.
There are headers left in webrtc/ with the same name including the
moved headers in webrtc/call/ for not breaking external projects
depending on WebRTC.
At the same time, some minor cleanup is done: Non-pure-virtual functions declared in the two affected headers now have definitions in the same target. After making this change, our 'chromium-style' plugin detected some style violations that have now been fixed: non-inlined constructors and destructors have been added to a number of classes, both inside the GN target of the two affected headers, and in other targets.
BUG=webrtc:8107
Review-Url: https://codereview.webrtc.org/3000253002
Cr-Commit-Position: refs/heads/master@{#19448}
diff --git a/webrtc/media/BUILD.gn b/webrtc/media/BUILD.gn
index be75f6f..dcb5247 100644
--- a/webrtc/media/BUILD.gn
+++ b/webrtc/media/BUILD.gn
@@ -93,6 +93,11 @@
"base/videocommon.h",
"base/videosourcebase.cc",
"base/videosourcebase.h",
+
+ # TODO(aleloi): add "base/videosinkinterface.h"
+ "base/videosourceinterface.cc",
+
+ # TODO(aleloi): add "base/videosourceinterface.h"
]
if (!build_with_chromium && is_clang) {
@@ -217,7 +222,6 @@
}
deps += [
":rtc_media_base",
- "..:video_stream_api",
"..:webrtc_common",
"../api:call_api",
"../api:libjingle_peerconnection_api",
@@ -228,6 +232,7 @@
"../api/audio_codecs:builtin_audio_encoder_factory",
"../api/video_codecs:video_codecs_api",
"../call",
+ "../call:video_stream_api",
"../common_video:common_video",
"../modules/audio_coding:rent_a_codec",
"../modules/audio_device:audio_device",
@@ -322,7 +327,7 @@
include_dirs = []
public_deps = []
deps = [
- "..:video_stream_api",
+ "../call:video_stream_api",
"../modules/audio_coding:rent_a_codec",
"../modules/audio_processing:audio_processing",
"../modules/rtp_rtcp:rtp_rtcp",
