WebRTC: Implement MediaEndpointPeerConnection::stop()
authoradam.bergkvist@ericsson.com <adam.bergkvist@ericsson.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Oct 2016 18:56:43 +0000 (18:56 +0000)
committeradam.bergkvist@ericsson.com <adam.bergkvist@ericsson.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Oct 2016 18:56:43 +0000 (18:56 +0000)
https://bugs.webkit.org/show_bug.cgi?id=163660

Reviewed by Philippe Normand.

Make MediaEndpointPeerConnection::stop() stop its MediaEndpoint.

Testing: This fix deals with cleanup and tearing down resources down in the platform layer
and is not covered by automated tests. It is however observable in the manual test added
in [1] where the remote videos should stop when the connections are closed.

[1] http://webkit.org/b/163327

* Modules/mediastream/MediaEndpointPeerConnection.cpp:
(WebCore::MediaEndpointPeerConnection::stop):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@207552 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/Modules/mediastream/MediaEndpointPeerConnection.cpp

index b055534..7c042fb 100644 (file)
@@ -1,3 +1,21 @@
+2016-10-19  Adam Bergkvist  <adam.bergkvist@ericsson.com>
+
+        WebRTC: Implement MediaEndpointPeerConnection::stop()
+        https://bugs.webkit.org/show_bug.cgi?id=163660
+
+        Reviewed by Philippe Normand.
+
+        Make MediaEndpointPeerConnection::stop() stop its MediaEndpoint.
+
+        Testing: This fix deals with cleanup and tearing down resources down in the platform layer
+        and is not covered by automated tests. It is however observable in the manual test added
+        in [1] where the remote videos should stop when the connections are closed.
+
+        [1] http://webkit.org/b/163327
+
+        * Modules/mediastream/MediaEndpointPeerConnection.cpp:
+        (WebCore::MediaEndpointPeerConnection::stop):
+
 2016-10-19  Dave Hyatt  <hyatt@apple.com>
 
         [CSS Parser] Fix -webkit-mask-box-image parsing
index f0ba4e2..3ce3f29 100644 (file)
@@ -745,7 +745,7 @@ void MediaEndpointPeerConnection::replaceTrackTask(RTCRtpSender& sender, const S
 
 void MediaEndpointPeerConnection::stop()
 {
-    notImplemented();
+    m_mediaEndpoint->stop();
 }
 
 void MediaEndpointPeerConnection::markAsNeedingNegotiation()