Enable keyboard navigation for Desktopcapture window selection dialog

chrome/browser/ui/views/desktop_media_picker_views.h

Index: chrome/browser/ui/views/desktop_media_picker_views.h
diff --git a/chrome/browser/ui/views/desktop_media_picker_views.h b/chrome/browser/ui/views/desktop_media_picker_views.h
index 3552dba5bae55fecade62173e82c1eb1e2a9ae43..fce7959034285776d88a639a7796e1762f010cbc 100644
--- a/chrome/browser/ui/views/desktop_media_picker_views.h
+++ b/chrome/browser/ui/views/desktop_media_picker_views.h
@@ -129,6 +129,9 @@ class DesktopMediaPickerDialogView : public views::DialogDelegateView {
   gfx::Size GetPreferredSize() const override;
   void Layout() override;
 
+  // views::DialogDelegate overrides.
+  views::View* GetInitiallyFocusedView() override;

[Peter Kasting, 2015/09/24 04:10:25]

You don't directly subclass DialogDelegate; you get this function through
DialogDelegateView, and that's the only override section you should have.  (The
existing separate section for View shouldn't exist either.)

Or to put it differently, this class doesn't care about how DialogDelegateView
is itself implemented, and if we were to change that someday to move this method
from one base class to another, we shouldn't need to check all subclasses (of
any depth) of all these classes to update the comments and organization of their
overrides.

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

Done.

+
   // views::DialogDelegateView overrides.
   ui::ModalType GetModalType() const override;
   base::string16 GetWindowTitle() const override;