From 43a723a71b3df5a54bfd1868283ce29226c50828 Mon Sep 17 00:00:00 2001 From: Steve Anton Date: Thu, 4 Jan 2018 15:48:17 -0800 Subject: [PATCH] Move AddTrack stats update to submethod 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 Commit-Queue: Steve Anton Cr-Commit-Position: refs/heads/master@{#21508} --- pc/peerconnection.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pc/peerconnection.cc b/pc/peerconnection.cc index 91cc2ff1a6..78fcf1d666 100644 --- a/pc/peerconnection.cc +++ b/pc/peerconnection.cc @@ -1013,7 +1013,6 @@ rtc::scoped_refptr 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; }