Bug 1208371 - Remove unused MediaManager::NotifyMediaStreamTrackEnded. r=jib
authorAndreas Pehrson <pehrsons@gmail.com>
Tue, 05 Jan 2016 10:16:21 +0800
changeset 348403 aeffc7ef6fea01e6b6f99f665dc75864e4e00d6e
parent 348402 0e67203c73ffac63d0a4b9172bb95dd4977220ab
child 348404 987bb0e75a341e0d0968520ee2c47cf06eda4395
push id14828
push userpehrsons@gmail.com
push dateThu, 07 Apr 2016 12:57:27 +0000
reviewersjib
bugs1208371
milestone48.0a1
Bug 1208371 - Remove unused MediaManager::NotifyMediaStreamTrackEnded. r=jib MozReview-Commit-ID: 3QFbTliYTPw
dom/media/MediaManager.cpp
--- a/dom/media/MediaManager.cpp
+++ b/dom/media/MediaManager.cpp
@@ -757,31 +757,16 @@ public:
     p->Then([promise](bool& aDummy) mutable {
       promise->MaybeResolve(false);
     }, [promise](MediaStreamError*& reason) mutable {
       promise->MaybeReject(reason);
     });
     return promise.forget();
   }
 
-#if 0
-  virtual void NotifyMediaStreamTrackEnded(dom::MediaStreamTrack* aTrack)
-  {
-    TrackID trackID = aTrack->GetTrackID();
-    // We override this so we can also tell the backend to stop capturing if the track ends
-    LOG(("track %d ending, type = %s",
-         trackID, aTrack->AsAudioStreamTrack() ? "audio" : "video"));
-    MOZ_ASSERT(aTrack->AsVideoStreamTrack() || aTrack->AsAudioStreamTrack());
-    mListener->StopTrack(trackID, !!aTrack->AsAudioStreamTrack());
-
-    // forward to superclass
-    DOMLocalMediaStream::NotifyMediaStreamTrackEnded(aTrack);
-  }
-#endif
-
   // Allow getUserMedia to pass input data directly to PeerConnection/MediaPipeline
   bool AddDirectListener(MediaStreamDirectListener *aListener) override
   {
     if (GetSourceStream()) {
       GetSourceStream()->AddDirectListener(aListener);
       return true; // application should ignore NotifyQueuedTrackData
     }
     return false;