diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-29 17:40:13 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-29 17:40:13 +0000 |
| commit | cec6c9c550f44741c7acacfd5d3f67e919110c4c (patch) | |
| tree | f6778f4a10b1aa5b0a7b74526c632377214703b1 /views/view.cc | |
| parent | 210796e7c8207e56dae29392b62e1e79ec9f837f (diff) | |
| download | chromium_src-cec6c9c550f44741c7acacfd5d3f67e919110c4c.zip chromium_src-cec6c9c550f44741c7acacfd5d3f67e919110c4c.tar.gz chromium_src-cec6c9c550f44741c7acacfd5d3f67e919110c4c.tar.bz2 | |
See original review at
http://codereview.chromium.org/251001/show
This CL hooks the focus traversal for the TabbedPaneView on Linux with toolkit views.
It also has few focus related fixed (focus with buttons, clearing focus effectively in the focus manager).
BUG=None
TEST=Run the view_examples app. Press tab to move the focus. Try the different tabs.
Review URL: http://codereview.chromium.org/246030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27504 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/view.cc')
| -rw-r--r-- | views/view.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/views/view.cc b/views/view.cc index c5db5db..3c5d77b 100644 --- a/views/view.cc +++ b/views/view.cc @@ -277,11 +277,11 @@ bool View::HasFocus() { } void View::Focus() { - // Set the native focus to the root view window so it receives the keyboard - // messages. + // By default, we clear the native focus. This ensures that no visible native + // view as the focus and that we still receive keyboard inputs. FocusManager* focus_manager = GetFocusManager(); if (focus_manager) - focus_manager->FocusNativeView(GetRootView()->GetWidget()->GetNativeView()); + focus_manager->ClearNativeFocus(); } void View::SetHotTracked(bool flag) { |