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..7224aab43e1995d3230a2d41dfa5aa540ef527e1 100644
--- a/chrome/browser/ui/views/desktop_media_picker_views.cc
+++ b/chrome/browser/ui/views/desktop_media_picker_views.cc
@@ -443,6 +443,10 @@ DesktopMediaPickerDialogView::DesktopMediaPickerDialogView(
   }
 
   list_view_->StartUpdating(dialog_window_id);
+
+  // Enable and request keyboard focus for list view.
+  list_view_->SetFocusable(true);

[Peter Kasting, 2015/09/23 20:54:20]

I would move this to the list view's constructor.

[gyzhou, 2015/09/23 22:37:26]

Done.

+  list_view_->RequestFocus();

[Peter Kasting, 2015/09/23 20:54:20]

I would instead implement this by overriding GetInitiallyFocusedView() to return
the list view.

[gyzhou, 2015/09/23 22:37:26]

Done.

 }
 
 DesktopMediaPickerDialogView::~DesktopMediaPickerDialogView() {}