Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(431)

Unified Diff: webrtc/api/rtpsender.cc

Issue 2023373002: Separating internal and external methods of RtpSender/RtpReceiver. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/api/rtpsender.cc
diff --git a/webrtc/api/rtpsender.cc b/webrtc/api/rtpsender.cc
index 360b6868fd377b99e1f64fdeb21bead5d7b490df..3ca6e8c16f07433a422d8b44ab2252efb445b384 100644
--- a/webrtc/api/rtpsender.cc
+++ b/webrtc/api/rtpsender.cc
@@ -145,6 +145,15 @@ bool AudioRtpSender::SetTrack(MediaStreamTrackInterface* track) {
return true;
}
+RtpParameters AudioRtpSender::GetParameters() const {
+ return provider_->GetAudioRtpParameters(ssrc_);
+}
+
+bool AudioRtpSender::SetParameters(const RtpParameters& parameters) {
+ TRACE_EVENT0("webrtc", "AudioRtpSender::SetParameters");
+ return provider_->SetAudioRtpParameters(ssrc_, parameters);
+}
+
void AudioRtpSender::SetSsrc(uint32_t ssrc) {
TRACE_EVENT0("webrtc", "AudioRtpSender::SetSsrc");
if (stopped_ || ssrc == ssrc_) {
@@ -207,15 +216,6 @@ void AudioRtpSender::SetAudioSend() {
provider_->SetAudioSend(ssrc_, track_->enabled(), options, source);
}
-RtpParameters AudioRtpSender::GetParameters() const {
- return provider_->GetAudioRtpParameters(ssrc_);
-}
-
-bool AudioRtpSender::SetParameters(const RtpParameters& parameters) {
- TRACE_EVENT0("webrtc", "AudioRtpSender::SetParameters");
- return provider_->SetAudioRtpParameters(ssrc_, parameters);
-}
-
VideoRtpSender::VideoRtpSender(VideoTrackInterface* track,
const std::string& stream_id,
VideoProviderInterface* provider)
@@ -305,6 +305,15 @@ bool VideoRtpSender::SetTrack(MediaStreamTrackInterface* track) {
return true;
}
+RtpParameters VideoRtpSender::GetParameters() const {
+ return provider_->GetVideoRtpParameters(ssrc_);
+}
+
+bool VideoRtpSender::SetParameters(const RtpParameters& parameters) {
+ TRACE_EVENT0("webrtc", "VideoRtpSender::SetParameters");
+ return provider_->SetVideoRtpParameters(ssrc_, parameters);
+}
+
void VideoRtpSender::SetSsrc(uint32_t ssrc) {
TRACE_EVENT0("webrtc", "VideoRtpSender::SetSsrc");
if (stopped_ || ssrc == ssrc_) {
@@ -349,13 +358,4 @@ void VideoRtpSender::SetVideoSend() {
provider_->SetVideoSend(ssrc_, track_->enabled(), &options);
}
-RtpParameters VideoRtpSender::GetParameters() const {
- return provider_->GetVideoRtpParameters(ssrc_);
-}
-
-bool VideoRtpSender::SetParameters(const RtpParameters& parameters) {
- TRACE_EVENT0("webrtc", "VideoRtpSender::SetParameters");
- return provider_->SetVideoRtpParameters(ssrc_, parameters);
-}
-
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698