Bug 1213441 - Remove pc.removeStream properly. r=jesup,smaug

MozReview-Commit-ID: CvmDVGUVMN2

--HG--
extra : rebase_source : 5bd3a0f17af4160d714cf090bbc4131c22e2d4f5

dom/media/PeerConnection.js

@@ -1033,12 +1033,6 @@ RTCPeerConnection.prototype = {
     stream.getTracks().forEach(track => this.addTrack(track, stream));
   },
 
-  removeStream: function(stream) {
-     // Bug 844295: Not implementing this functionality.
-     throw new this._win.DOMException("removeStream not yet implemented",
-                                      "NotSupportedError");
-  },
-
   getStreamById: function(id) {
     throw new this._win.DOMException("getStreamById not yet implemented",
                                      "NotSupportedError");

dom/media/tests/mochitest/test_peerConnection_close.html

@@ -107,10 +107,6 @@
         pc.addStream("Invalid Media Stream")},
         "addStream() on closed PC raised expected exception");
 
-      SimpleTest.doesThrow(function() {
-        pc.removeStream("Invalid Media Stream")},
-        "removeStream() on closed PC raised expected exception");
-
       SimpleTest.doesThrow(function() {
         pc.createDataChannel({})},
         "createDataChannel() on closed PC raised expected exception");

dom/webidl/RTCPeerConnection.webidl

@@ -123,7 +123,6 @@ interface RTCPeerConnection : EventTarget  {
   [UnsafeInPrerendering]
   MediaStream? getStreamById (DOMString streamId);
   void addStream (MediaStream stream);
-  void removeStream (MediaStream stream);
 
   // replaces addStream; fails if already added
   // because a track can be part of multiple streams, stream parameters