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

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

Issue 108063004: Give up focus if the focused view becomes unfocusable (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Extracted to common code into FocusManager class Created 7 years 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/fullscreen_exit_bubble_views.cc
diff --git a/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc b/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc
index 7e0f83832f30a68047a26b34d137eca0fcc74ce6..6e66fee6d0aaa968eeec40c50fe461a8cac1cf4b 100644
--- a/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc
+++ b/chrome/browser/ui/views/fullscreen_exit_bubble_views.cc
@@ -66,12 +66,12 @@ ButtonView::ButtonView(views::ButtonListener* listener,
deny_button_(NULL) {
accept_button_ = new views::LabelButton(listener, string16());
accept_button_->SetStyle(views::Button::STYLE_NATIVE_TEXTBUTTON);
- accept_button_->set_focusable(false);
+ accept_button_->SetFocusable(false);
AddChildView(accept_button_);
deny_button_ = new views::LabelButton(listener, string16());
deny_button_->SetStyle(views::Button::STYLE_NATIVE_TEXTBUTTON);
- deny_button_->set_focusable(false);
+ deny_button_->SetFocusable(false);
AddChildView(deny_button_);
SetLayoutManager(new views::BoxLayout(views::BoxLayout::kHorizontal, 0, 0,
@@ -138,7 +138,7 @@ FullscreenExitBubbleViews::FullscreenExitView::FullscreenExitView(
SK_ColorWHITE);
set_background(new views::BubbleBackground(bubble_border));
set_border(bubble_border);
- set_focusable(false);
+ SetFocusable(false);
ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
message_label_ = new views::Label();
@@ -152,7 +152,7 @@ FullscreenExitBubbleViews::FullscreenExitView::FullscreenExitView(
link_ = new views::Link();
link_->set_collapse_when_hidden(true);
- link_->set_focusable(false);
+ link_->SetFocusable(false);
#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