Changeset 213166 in webkit

Timestamp:

Feb 28, 2017 1:02:25 PM (7 years ago)

Author:

commit-queue@webkit.org

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 <​youenn@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.

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• NetworkProcess/webrtc/NetworkRTCProvider.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2017-02-28  Youenn Fablet  <youenn@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  <mrajca@apple.com>
 

trunk/Source/WebKit2/NetworkProcess/webrtc/NetworkRTCProvider.cpp

@@ -66 +66 @@
     callOnRTCNetworkThread([this]() {
         m_sockets.clear();
+        callOnMainThread([provider = makeRef(*this)]() {
+            if (provider->m_rtcNetworkThread)
+                provider->m_rtcNetworkThread->Stop();
+        });
     });
 }