diff options
| author | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-06 07:07:18 +0000 |
|---|---|---|
| committer | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-06 07:07:18 +0000 |
| commit | 6a054ffaae82f5ac8d6b876d7c85f0d87d892e42 (patch) | |
| tree | 86b4d5a0c5386f3d50742af099b5009ad1dad14d /views/controls/native_control_win.cc | |
| parent | 6ca800b9b2618e6ae4b92e189a9b1ae73746e49f (diff) | |
| download | chromium_src-6a054ffaae82f5ac8d6b876d7c85f0d87d892e42.zip chromium_src-6a054ffaae82f5ac8d6b876d7c85f0d87d892e42.tar.gz chromium_src-6a054ffaae82f5ac8d6b876d7c85f0d87d892e42.tar.bz2 | |
Improvements to accessibility extension api support for "views":
1. Handles the new wrench menu.
2. Uses NotifyAccessibilityEvent to find out when focus changes, rather
than installing focus change listeners that need to be cleaned up.
BUG=none
TEST=Updated AccessibilityEventRouterViewsTest.TestFocusNotification
Review URL: http://codereview.chromium.org/3056045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55196 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/controls/native_control_win.cc')
| -rw-r--r-- | views/controls/native_control_win.cc | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/views/controls/native_control_win.cc b/views/controls/native_control_win.cc index 6d23a5c..ba27f85 100644 --- a/views/controls/native_control_win.cc +++ b/views/controls/native_control_win.cc @@ -102,11 +102,15 @@ void NativeControlWin::Focus() { View* parent_view = GetParent(); // Due to some controls not behaving as expected without having - // a native win32 control, we exclude the following from sending - // their IAccessible as focus events. - if (parent_view->GetClassName() != views::Combobox::kViewClassName && - parent_view->HasFocus()) - parent_view->NotifyAccessibilityEvent(AccessibilityTypes::EVENT_FOCUS); + // a native win32 control, we don't always send a native (MSAA) + // focus notification. + bool send_native_event = + parent_view->GetClassName() != views::Combobox::kViewClassName && + parent_view->HasFocus(); + + // Send the accessibility focus notification. + parent_view->NotifyAccessibilityEvent(AccessibilityTypes::EVENT_FOCUS, + send_native_event); } //////////////////////////////////////////////////////////////////////////////// |