Web Inspector: dock buttons disappear if Web Inspector goes fullscreen
authorbburg@apple.com <bburg@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 22 Sep 2018 00:14:05 +0000 (00:14 +0000)
committerbburg@apple.com <bburg@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 22 Sep 2018 00:14:05 +0000 (00:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=189865
<rdar://problem/42600534>

Reviewed by Matt Baker.

When a fullscreen Safari tab has a docked Inspector, and the "detach" button
is clicked, Inspector goes into its own fullscreen window. In that window,
there are no buttons for docking to side or bottom.

It turns out that we always send setDockingUnavailable(true) if the inspector
window is fullscreen. This eventually causes the inspector to reflect that
by hiding the docking-related buttons.

* UIProcess/mac/WebInspectorProxyMac.mm:
(WebKit::WebInspectorProxy::platformCanAttach):
This early exit does not seem to serve any purpose, so remove it.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/mac/WebInspectorProxyMac.mm

index 586f694..29a63b6 100644 (file)
@@ -1,3 +1,23 @@
+2018-09-21  Brian Burg  <bburg@apple.com>
+
+        Web Inspector: dock buttons disappear if Web Inspector goes fullscreen
+        https://bugs.webkit.org/show_bug.cgi?id=189865
+        <rdar://problem/42600534>
+
+        Reviewed by Matt Baker.
+
+        When a fullscreen Safari tab has a docked Inspector, and the "detach" button
+        is clicked, Inspector goes into its own fullscreen window. In that window,
+        there are no buttons for docking to side or bottom.
+
+        It turns out that we always send setDockingUnavailable(true) if the inspector
+        window is fullscreen. This eventually causes the inspector to reflect that
+        by hiding the docking-related buttons.
+
+        * UIProcess/mac/WebInspectorProxyMac.mm:
+        (WebKit::WebInspectorProxy::platformCanAttach):
+        This early exit does not seem to serve any purpose, so remove it.
+
 2018-09-21  Chris Dumez  <cdumez@apple.com>
 
         Extending the lifetime of a NetworkProcessProxy / StorageProcessProxy may cause it to have a stale WebProcessPool pointer
index 87e02bf..f649fe8 100644 (file)
@@ -347,9 +347,6 @@ bool WebInspectorProxy::platformIsFront()
 
 bool WebInspectorProxy::platformCanAttach(bool webProcessCanAttach)
 {
-    if ([m_inspectorWindow styleMask] & NSWindowStyleMaskFullScreen)
-        return false;
-
     NSView *inspectedView = inspectedPage()->inspectorAttachmentView();
     if ([WKInspectorViewController viewIsInspectorWebView:inspectedView])
         return webProcessCanAttach;