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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/views/controls/scrollbar/native_scroll_bar_views.cc

Issue 1690543004: MacViews: Implement Full Keyboard Access. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments. Make patch smaller temporarily. Created 4 years, 9 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
« ui/views/controls/button/image_button.h ('K') | « ui/views/controls/link.cc ('k') | ui/views/controls/separator.cc » ('j') | ui/views/focus/focus_manager.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/controls/scrollbar/native_scroll_bar_views.cc
diff --git a/ui/views/controls/scrollbar/native_scroll_bar_views.cc b/ui/views/controls/scrollbar/native_scroll_bar_views.cc
index 3348b5bcbca041cc3e422722b685040a623e0c45..6b0fc03553189dc57b14affb43fdeff0878e36a6 100644
--- a/ui/views/controls/scrollbar/native_scroll_bar_views.cc
+++ b/ui/views/controls/scrollbar/native_scroll_bar_views.cc
@@ -72,8 +72,7 @@ class ScrollBarThumb : public BaseScrollBarThumb {
ScrollBarButton::ScrollBarButton(ButtonListener* listener, Type type)
: BaseScrollBarButton(listener),
type_(type) {
- SetFocusable(false);
- SetAccessibilityFocusable(false);
+ SetFocusBehavior(NEVER);
}
ScrollBarButton::~ScrollBarButton() {
@@ -150,8 +149,7 @@ ui::NativeTheme::State
ScrollBarThumb::ScrollBarThumb(BaseScrollBar* scroll_bar)
: BaseScrollBarThumb(scroll_bar),
scroll_bar_(scroll_bar) {
- SetFocusable(false);
- SetAccessibilityFocusable(false);
+ SetFocusBehavior(NEVER);
}
ScrollBarThumb::~ScrollBarThumb() {
« ui/views/controls/button/image_button.h ('K') | « ui/views/controls/link.cc ('k') | ui/views/controls/separator.cc » ('j') | ui/views/focus/focus_manager.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698