Web Inspector: UIProcess ASSERT in ~RemoteWebInspectorProxy when InspectorProcess...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Oct 2017 21:43:58 +0000 (21:43 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Oct 2017 21:43:58 +0000 (21:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=179044

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2017-10-31
Reviewed by Alex Christensen.

* UIProcess/RemoteWebInspectorProxy.cpp:
(WebKit::RemoteWebInspectorProxy::frontendDidClose):
Add a protector to keep the proxy alive for the duration of this method.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/RemoteWebInspectorProxy.cpp

index 45eebee..b35ac9e 100644 (file)
@@ -1,3 +1,14 @@
+2017-10-31  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: UIProcess ASSERT in ~RemoteWebInspectorProxy when InspectorProcess crashes
+        https://bugs.webkit.org/show_bug.cgi?id=179044
+
+        Reviewed by Alex Christensen.
+
+        * UIProcess/RemoteWebInspectorProxy.cpp:
+        (WebKit::RemoteWebInspectorProxy::frontendDidClose):
+        Add a protector to keep the proxy alive for the duration of this method.
+
 2017-10-31  Youenn Fablet  <youenn@apple.com>
 
         Crash in:  com.apple.WebKit: WebKit::CacheStorage::Caches::initializeSize(WTF::Function<void (std::optional<WebCore::DOMCacheEngine::Error>&&)>&&) + 30 (CacheStorageEngineCaches.cpp:163)
index 8e83895..2f1e711 100644 (file)
@@ -82,6 +82,8 @@ void RemoteWebInspectorProxy::sendMessageToFrontend(const String& message)
 
 void RemoteWebInspectorProxy::frontendDidClose()
 {
+    Ref<RemoteWebInspectorProxy> protect(*this);
+
     if (m_client)
         m_client->closeFromFrontend();