Title: [213166] trunk/Source/WebKit2
Revision
213166
Author
commit-qu...@webkit.org
Date
2017-02-28 13:02:25 -0800 (Tue, 28 Feb 2017)

Log Message

[WebRTC] NetworkRTCProvider should stop the network thread from the main thread
https://bugs.webkit.org/show_bug.cgi?id=168972

Patch by Youenn Fablet <you...@apple.com> on 2017-02-28
Reviewed by Alex Christensen.

* NetworkProcess/webrtc/NetworkRTCProvider.cpp:
(WebKit::NetworkRTCProvider::close): Ensuring to stop the network thread from the main thread when provider is
getting closed.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (213165 => 213166)


--- trunk/Source/WebKit2/ChangeLog	2017-02-28 20:57:35 UTC (rev 213165)
+++ trunk/Source/WebKit2/ChangeLog	2017-02-28 21:02:25 UTC (rev 213166)
@@ -1,3 +1,14 @@
+2017-02-28  Youenn Fablet  <you...@apple.com>
+
+        [WebRTC] NetworkRTCProvider should stop the network thread from the main thread
+        https://bugs.webkit.org/show_bug.cgi?id=168972
+
+        Reviewed by Alex Christensen.
+
+        * NetworkProcess/webrtc/NetworkRTCProvider.cpp:
+        (WebKit::NetworkRTCProvider::close): Ensuring to stop the network thread from the main thread when provider is
+        getting closed.
+
 2017-02-24  Matt Rajca  <mra...@apple.com>
 
         Media: notify clients when autoplayed media ends without being paused

Modified: trunk/Source/WebKit2/NetworkProcess/webrtc/NetworkRTCProvider.cpp (213165 => 213166)


--- trunk/Source/WebKit2/NetworkProcess/webrtc/NetworkRTCProvider.cpp	2017-02-28 20:57:35 UTC (rev 213165)
+++ trunk/Source/WebKit2/NetworkProcess/webrtc/NetworkRTCProvider.cpp	2017-02-28 21:02:25 UTC (rev 213166)
@@ -65,6 +65,10 @@
 
     callOnRTCNetworkThread([this]() {
         m_sockets.clear();
+        callOnMainThread([provider = makeRef(*this)]() {
+            if (provider->m_rtcNetworkThread)
+                provider->m_rtcNetworkThread->Stop();
+        });
     });
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to