DevTools: allow focus to be restored to the drawer when dialog closes

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

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 391 matching lines @@
     {
         return this._drawerSplitWidget.isSidebarMinimized();
     },
 
     /**
      * @override
      * @return {!Element}
      */
     defaultFocusedElement: function()
     {
-        return this._currentPanel ? this._currentPanel.defaultFocusedElement() : null;
+        if (this._drawer.hasFocus()) {

[lushnikov, 2016/05/23 18:19:24]

style 1: the if blocks with return statements should not have "else" statement
style 2: one-line blocks should be formatted without {}

if (this._drawer.hasFocus())
    return this._drawer.deafultFocusedElement();
return this._currentPanel ? this._currentPanel.defaultFocusedElement() : null;

[luoe, 2016/05/23 19:18:41]

Done.

+            return this._drawer.defaultFocusedElement();
+        } else {
+            return this._currentPanel ? this._currentPanel.defaultFocusedElement() : null;
+        }
     },
 
     _keyPress: function(event)
     {
         // BUG 104250: Windows 7 posts a WM_CHAR message upon the Ctrl+']' keypress.
         // Any charCode < 32 is not going to be a valid keypress.
         if (event.charCode < 32 && WebInspector.isWin())
             return;
         clearTimeout(this._keyDownTimer);
         delete this._keyDownTimer;
@@ skipping 177 matching lines @@
      */
     handleAction: function(context, actionId)
     {
         if (WebInspector.inspectorView.drawerVisible())
             WebInspector.inspectorView.closeDrawer();
         else
             WebInspector.inspectorView.showDrawer();
         return true;
     }
 }