admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Move AddTrack stats update to submethod

Browse Source 
AddTrack is just a legacy wrapper for the new AddTrack method, so
calling the new AddTrack method should do everything that the old one
does.

Bug: None
Change-Id: I272a9e9584c470d54243377c1307b786f41c660d
Reviewed-on: https://webrtc-review.googlesource.com/37546
Reviewed-by: Peter Thatcher <pthatcher@webrtc.org>
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#21508}
This commit is contained in:
Steve Anton 2018-01-04 15:48:17 -08:00 committed by Commit Bot
parent dc8b5ab350
commit 43a723a71b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pc/peerconnection.cc
View File

@ -1013,7 +1013,6 @@ rtc::scoped_refptr<RtpSenderInterface> PeerConnection::AddTrack(
if (!sender_or_error.ok()) {
return nullptr;
}
stats_->AddTrack(track);
return sender_or_error.MoveValue();
}
@ -1059,6 +1058,7 @@ PeerConnection::AddTrackWithStreamLabels(
: AddTrackPlanB(track, adjusted_stream_labels));
if (sender_or_error.ok()) {
observer_->OnRenegotiationNeeded();
stats_->AddTrack(track);
}
return sender_or_error;
}