Move focus to web contents when DownloadShelfView is closed

Currently, keyboard focus disappears if the DownloadShelfView is closed while it is focused. This change moves focus to the web contents when it is closing.

Bug: 846466
Change-Id: I2fd24d5e918a6be7ce46299b31b53d5fa92aaa97
Reviewed-on: https://chromium-review.googlesource.com/c/1302871
Commit-Queue: Collin Baker <collinbaker@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#603601}

chrome/browser/ui/views/download/download_shelf_view.cc

@@ -390,6 +390,16 @@ void DownloadShelfView::Closed() {
       ++i;
     }
   }
+
+  // If we had keyboard focus, calling SetVisible(false) causes keyboard focus
+  // to be completely lost. To prevent this, we focus another view: the web
+  // contents. TODO(collinbaker): https://crbug.com/846466 Fix
+  // AccessiblePaneView::SetVisible or FocusManager to make this unnecessary.
+  auto* focus_manager = GetFocusManager();
+  if (focus_manager && Contains(focus_manager->GetFocusedView())) {
+    get_parent()->contents_web_view()->RequestFocus();
+  }
+
   SetVisible(false);
 }