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

Unified Diff: chrome/browser/ui/views/exclusive_access_bubble_views.cc

Issue 1690543004: MacViews: Implement Full Keyboard Access. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months 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/exclusive_access_bubble_views.cc
diff --git a/chrome/browser/ui/views/exclusive_access_bubble_views.cc b/chrome/browser/ui/views/exclusive_access_bubble_views.cc
index fa106bf53b5056d40f21af9a73724729ee8dde75..ecdf2847c5fcf0a9db9981703c6e2e12198b19f2 100644
--- a/chrome/browser/ui/views/exclusive_access_bubble_views.cc
+++ b/chrome/browser/ui/views/exclusive_access_bubble_views.cc
@@ -74,12 +74,12 @@ ButtonView::ButtonView(views::ButtonListener* listener,
: accept_button_(nullptr), deny_button_(nullptr) {
accept_button_ = new views::LabelButton(listener, base::string16());
accept_button_->SetStyle(views::Button::STYLE_BUTTON);
- accept_button_->SetFocusable(false);
+ accept_button_->SetFocusBehavior(views::View::FocusBehavior::NEVER);
AddChildView(accept_button_);
deny_button_ = new views::LabelButton(listener, base::string16());
deny_button_->SetStyle(views::Button::STYLE_BUTTON);
- deny_button_->SetFocusable(false);
+ deny_button_->SetFocusBehavior(views::View::FocusBehavior::NEVER);
AddChildView(deny_button_);
SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0,
@@ -257,7 +257,7 @@ ExclusiveAccessBubbleViews::ExclusiveAccessView::ExclusiveAccessView(
views::BubbleBorder::NONE, shadow_type, background_color));
set_background(new views::BubbleBackground(bubble_border.get()));
SetBorder(std::move(bubble_border));
- SetFocusable(false);
+ SetFocusBehavior(views::View::FocusBehavior::NEVER);
ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
const gfx::FontList& font_list =
@@ -273,7 +273,7 @@ ExclusiveAccessBubbleViews::ExclusiveAccessView::ExclusiveAccessView(
foreground_color, background_color);
link_ = new views::Link();
- link_->SetFocusable(false);
+ link_->SetFocusBehavior(views::View::FocusBehavior::NEVER);
#if defined(OS_CHROMEOS)
// On CrOS, the link text doesn't change, since it doesn't show the shortcut.
link_->SetText(l10n_util::GetStringUTF16(IDS_EXIT_FULLSCREEN_MODE));

Powered by Google App Engine
This is Rietveld 408576698