DevTools: fix deepActiveElement and focus on Dialog when document blurs

third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js

Index: third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js b/third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js
index 231d4de44ed8d2905b3569ca9fa89933cd22d38c..1b577335e2cc8c9cd9f29dad7b6cb80887329b2e 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js
@@ -876,6 +876,9 @@ WebInspector.setCurrentFocusElement = function(x)
         }
     } else if (WebInspector._previousFocusElement)
         WebInspector._previousFocusElement.blur();
+
+    if (!x)
+        WebInspector.Dialog.focus();

[dgozman, 2016/06/29 22:19:15]

I wonder how the dialog is different from document? Why do be respect the blur
on document, but not dialog?

[luoe, 2016/06/30 00:03:16]

When the document is blurred (e.g. clicking on a non-focusable element),
document.activeElement can be == document.body.  So if the body has focus when a
dialog is open and 'Esc' is pressed, Dialog's onKeyDown won't fire and it won't
close.

I don't have a great reason why this is the right way to make sure 'Esc' always
works.  Maybe a better fix is to have Dialogs add an event listener to the
ownerDocument.body, just like how StylesSidebarPane does it.

 }
 
 WebInspector.restoreFocusFromElement = function(element)