Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(543)

Unified Diff: chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.h

Issue 689833004: Auto-focus the first button in the App Info dialog (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@accessibility_dialog
Patch Set: Made the initially focused view return NULL Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.h
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.h b/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.h
index 704ddde77591c6970918b9e500370a84695a7609..0a03082a268e267927ce0a27c61ec624704e1847 100644
--- a/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.h
+++ b/chrome/browser/ui/views/apps/app_info_dialog/app_info_footer_panel.h
@@ -44,8 +44,9 @@ class AppInfoFooterPanel
void LayoutButtons();
// Updates the visibility of the pin/unpin buttons so that only one is visible
- // at a time.
- void UpdatePinButtons();
+ // at a time. If |focus_button| is true, sets the focus to whichever button is
+ // now visible.
+ void UpdatePinButtons(bool focus_visible_button);
// Overridden from views::ButtonListener:
virtual void ButtonPressed(views::Button* sender,

Powered by Google App Engine
This is Rietveld 408576698