Enable keyboard navigation for Desktopcapture window selection dialog

chrome/browser/ui/views/desktop_media_picker_views.cc

Index: chrome/browser/ui/views/desktop_media_picker_views.cc
diff --git a/chrome/browser/ui/views/desktop_media_picker_views.cc b/chrome/browser/ui/views/desktop_media_picker_views.cc
index b2975b2b2309b25d1a1208e10f30ccd1bbb3a35c..7597ec41fa932922976c8e4408ab580e0e932265 100644
--- a/chrome/browser/ui/views/desktop_media_picker_views.cc
+++ b/chrome/browser/ui/views/desktop_media_picker_views.cc
@@ -206,6 +206,7 @@ DesktopMediaListView::DesktopMediaListView(
       media_list_(media_list.Pass()),
       weak_factory_(this) {
   media_list_->SetThumbnailSize(gfx::Size(kThumbnailWidth, kThumbnailHeight));
+  SetFocusable(true);
 }
 
 DesktopMediaListView::~DesktopMediaListView() {}
@@ -443,6 +444,8 @@ DesktopMediaPickerDialogView::DesktopMediaPickerDialogView(
   }
 
   list_view_->StartUpdating(dialog_window_id);
+
+  GetInitiallyFocusedView()->RequestFocus();

[Peter Kasting, 2015/09/24 20:46:56]

I've asked several times why this code is still here and you repeatedly haven't
answered and haven't removed it.  I don't understand why this is happening.

To reiterate what has already been explained: the whole point of using
GetInitiallyFocusedView() is to let the widget machinery focus the view for you.
 You should not be doing this yourself.  If you need to, something else is
broken elsewhere and this still isn't the right fix.  If you still don't
understand something, ask more questions about it, but don't let my feedback
simply go ignored.

[gyzhou, 2015/10/02 18:51:26]

Done.

 }
 
 DesktopMediaPickerDialogView::~DesktopMediaPickerDialogView() {}
@@ -495,6 +498,10 @@ bool DesktopMediaPickerDialogView::IsDialogButtonEnabled(
   return true;
 }
 
+views::View* DesktopMediaPickerDialogView::GetInitiallyFocusedView(){
+  return list_view_;
+}
+
 base::string16 DesktopMediaPickerDialogView::GetDialogButtonLabel(
     ui::DialogButton button) const {
   return l10n_util::GetStringUTF16(button == ui::DIALOG_BUTTON_OK ?