Bug 1437832 - Part 4: Remove unused function from PeerConnectionImpl r=jib

MozReview-Commit-ID: DJPLOWdm2IA

--HG--
extra : rebase_source : 74755f78b68fe04f413fb5ac0e68c5025b0270e1

media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp

@@ -2321,41 +2321,6 @@ PeerConnectionImpl::DisablePacketDump(unsigned long level,
   return NS_OK;
 }
 
-NS_IMETHODIMP
-PeerConnectionImpl::RemoveTrack(MediaStreamTrack& aTrack) {
-  PC_AUTO_ENTER_API_CALL(true);
-
-  std::vector<RefPtr<TransceiverImpl>>& transceivers =
-    mMedia->GetTransceivers();
-
-  nsresult rv = NS_ERROR_INVALID_ARG;
-
-  for (RefPtr<TransceiverImpl>& transceiver : transceivers) {
-    if (transceiver->HasSendTrack(&aTrack)) {
-      // TODO(bug 1401983): Move DTMF stuff to TransceiverImpl
-      for (size_t i = 0; i < mDTMFStates.Length(); ++i) {
-        if (mDTMFStates[i]->mTransceiver.get() == transceiver.get()) {
-          mDTMFStates[i]->mSendTimer->Cancel();
-          mDTMFStates.RemoveElementAt(i);
-          break;
-        }
-      }
-
-      rv = transceiver->UpdateSendTrack(nullptr);
-      break;
-    }
-  }
-
-  if (NS_FAILED(rv)) {
-    CSFLogError(LOGTAG, "Error updating send track on transceiver");
-    return rv;
-  }
-
-  aTrack.RemovePrincipalChangeObserver(this);
-
-  return NS_OK;
-}
-
 static int GetDTMFToneCode(uint16_t c)
 {
   const char* DTMF_TONECODES = "0123456789*#ABCD";

media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.h

@@ -393,12 +393,6 @@ public:
     rv = CloseStreams();
   }
 
-  NS_IMETHODIMP_TO_ERRORRESULT(RemoveTrack, ErrorResult &rv,
-                               mozilla::dom::MediaStreamTrack& aTrack)
-  {
-    rv = RemoveTrack(aTrack);
-  }
-
   already_AddRefed<TransceiverImpl> CreateTransceiverImpl(
       const nsAString& aKind,
       dom::MediaStreamTrack* aSendTrack,