From 68ded2fc5151bfc018779d6ad2db883d5d8ce8ba Mon Sep 17 00:00:00 2001 From: "Byron Campen [:bwc]" Date: Mon, 27 Apr 2020 14:38:26 +0000 Subject: [PATCH] Bug 1629565: Some logging that was helpful. r=mjf Differential Revision: https://phabricator.services.mozilla.com/D72243 --- media/mtransport/nricemediastream.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/media/mtransport/nricemediastream.cpp b/media/mtransport/nricemediastream.cpp index ff35114ceaa1..b94648ece19a 100644 --- a/media/mtransport/nricemediastream.cpp +++ b/media/mtransport/nricemediastream.cpp @@ -212,14 +212,19 @@ nsresult NrIceMediaStream::ConnectToPeer( MOZ_ASSERT(stream_); if (Matches(old_stream_, ufrag, pwd)) { - // Roll back to old stream, since we apparently aren't using the new one // (We swap before we close so we never have stream_ == nullptr) + MOZ_MTLOG(ML_DEBUG, + "Rolling back to old stream ufrag=" << ufrag << " " << name_); std::swap(stream_, old_stream_); CloseStream(&old_stream_); } else if (old_stream_) { // Right now we wait for ICE to complete before closing the old stream. // It might be worth it to close it sooner, but we don't want to close it // right away. + MOZ_MTLOG(ML_DEBUG, + "ICE restart committed, marking old stream as obsolete, " + "beginning switchover to ufrag=" + << ufrag << " " << name_); nr_ice_media_stream_set_obsolete(old_stream_); }