Unreviewed follow-up to r238330.
authorjoepeck@webkit.org <joepeck@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Nov 2018 03:06:38 +0000 (03:06 +0000)
committerjoepeck@webkit.org <joepeck@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Nov 2018 03:06:38 +0000 (03:06 +0000)
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::close):
Don't clear the pointer after invalidating since the pointer is
used later one. We don't mind it being used after invalidation
it will do the right thing, we'd rather have the WebInspectorProxy
lifetime match the WebPageProxy.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/WebPageProxy.cpp

index ca19231..fae0b09 100644 (file)
@@ -1,3 +1,14 @@
+2018-11-16  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Unreviewed follow-up to r238330.
+
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::close):
+        Don't clear the pointer after invalidating since the pointer is
+        used later one. We don't mind it being used after invalidation
+        it will do the right thing, we'd rather have the WebInspectorProxy
+        lifetime match the WebPageProxy.
+
 2018-11-16  Chris Dumez  <cdumez@apple.com>
 
         ProcessSwap.NavigationWithLockedHistoryWithoutPSON API test is a flaky crash in debug
index caab5e6..9e56c75 100644 (file)
@@ -941,7 +941,6 @@ void WebPageProxy::close()
 #endif
 
     m_inspector->invalidate();
-    m_inspector = nullptr;
 
     m_backForwardList->pageClosed();
     m_inspectorController->pageClosed();