Title: [199517] trunk/Source/WebKit2
Revision
199517
Author
beid...@apple.com
Date
2016-04-13 15:45:21 -0700 (Wed, 13 Apr 2016)

Log Message

Modern IDB: NetworkProcessConnection::didClose needs to have a self ref.
<rdar://problem/25700864> and https://bugs.webkit.org/show_bug.cgi?id=156559

Reviewed by Alex Christensen.

* WebProcess/Network/NetworkProcessConnection.cpp:
(WebKit::NetworkProcessConnection::didClose): Self ref to protect this.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (199516 => 199517)


--- trunk/Source/WebKit2/ChangeLog	2016-04-13 22:11:46 UTC (rev 199516)
+++ trunk/Source/WebKit2/ChangeLog	2016-04-13 22:45:21 UTC (rev 199517)
@@ -1,3 +1,13 @@
+2016-04-13  Brady Eidson  <beid...@apple.com>
+
+        Modern IDB: NetworkProcessConnection::didClose needs to have a self ref.
+        <rdar://problem/25700864> and https://bugs.webkit.org/show_bug.cgi?id=156559
+
+        Reviewed by Alex Christensen.
+
+        * WebProcess/Network/NetworkProcessConnection.cpp:
+        (WebKit::NetworkProcessConnection::didClose): Self ref to protect this.
+
 2016-04-13  Daniel Bates  <daba...@apple.com>
 
         REGRESSION (r199401): Internal builds of Safari hang on launch

Modified: trunk/Source/WebKit2/WebProcess/Network/NetworkProcessConnection.cpp (199516 => 199517)


--- trunk/Source/WebKit2/WebProcess/Network/NetworkProcessConnection.cpp	2016-04-13 22:11:46 UTC (rev 199516)
+++ trunk/Source/WebKit2/WebProcess/Network/NetworkProcessConnection.cpp	2016-04-13 22:45:21 UTC (rev 199517)
@@ -71,6 +71,7 @@
 void NetworkProcessConnection::didClose(IPC::Connection&)
 {
     // The NetworkProcess probably crashed.
+    Ref<NetworkProcessConnection> protector(*this);
     WebProcess::singleton().networkProcessConnectionClosed(this);
 
     Vector<String> dummyFilenames;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to