REGRESSION(r218089): [GTK] webkit_web_view_get_inspector() needs to be called, otherw...
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Mar 2018 06:37:41 +0000 (06:37 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Mar 2018 06:37:41 +0000 (06:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=181126

Reviewed by Carlos Alberto Lopez Perez.

Fix the condition to decide whether to detach the inspector view ourselves.

* UIProcess/gtk/WebInspectorProxyGtk.cpp:
(WebKit::WebInspectorProxy::platformDetach): Remove the inspector view from its parent if we don't have a client
or the client didn't detach it.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/gtk/WebInspectorProxyGtk.cpp

index 20e9845..5abf23f 100644 (file)
@@ -1,5 +1,18 @@
 2018-03-07  Carlos Garcia Campos  <cgarcia@igalia.com>
 
+        REGRESSION(r218089): [GTK] webkit_web_view_get_inspector() needs to be called, otherwise inspector does not close
+        https://bugs.webkit.org/show_bug.cgi?id=181126
+
+        Reviewed by Carlos Alberto Lopez Perez.
+
+        Fix the condition to decide whether to detach the inspector view ourselves.
+
+        * UIProcess/gtk/WebInspectorProxyGtk.cpp:
+        (WebKit::WebInspectorProxy::platformDetach): Remove the inspector view from its parent if we don't have a client
+        or the client didn't detach it.
+
+2018-03-07  Carlos Garcia Campos  <cgarcia@igalia.com>
+
         [GTK][WPE] Leak checker is not working in WebKitGLib web process tests
         https://bugs.webkit.org/show_bug.cgi?id=183404
 
index 78be087..49aecbe 100644 (file)
@@ -396,7 +396,7 @@ void WebInspectorProxy::platformDetach()
         return;
 
     GRefPtr<GtkWidget> inspectorView = m_inspectorView;
-    if (m_client && !m_client->detach(*this)) {
+    if (!m_client || !m_client->detach(*this)) {
         // Detach is called when m_isAttached is true, but it could called before
         // the inspector is opened if the inspector is shown/closed quickly. So,
         // we might not have a parent yet.