diff options
| author | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-30 05:47:34 +0000 |
|---|---|---|
| committer | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-30 05:47:34 +0000 |
| commit | 678e136687693d38e647c2795a909967a01ec014 (patch) | |
| tree | a96e5b22bba08946cb01972442679d2ef727a5b3 /content/browser/accessibility/browser_accessibility_manager_win.cc | |
| parent | d38a41f48cf338a1c74d58f883fc738d60fdaeae (diff) | |
| download | chromium_src-678e136687693d38e647c2795a909967a01ec014.zip chromium_src-678e136687693d38e647c2795a909967a01ec014.tar.gz chromium_src-678e136687693d38e647c2795a909967a01ec014.tar.bz2 | |
RWHI should implement BrowserAccessibilityDelegate
This is just refactoring, there should be no behavior changes.
I realized that most of the functionality of
BrowserAccessibilityDelegate was implemented by
RenderWidgetHostImpl, but each RenderWidgetHostView*
was actually implementing the interface, resulting in
extra code. It's simpler to have RWHI implement it and
just call into RWHV for the few cases where the view is
involved.
This also removes the parallel mac-only delegate, since
it was duplicating most of the same functionality.
Calling OriginInScreen is a couple of extra lines of code
because it needs to convert from NSPoint & NSSize to
gfx::Rect, but in every other case it's the same or simpler
this way.
BUG=none
NOTRY=true
Review URL: https://codereview.chromium.org/246773008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267097 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/accessibility/browser_accessibility_manager_win.cc')
| -rw-r--r-- | content/browser/accessibility/browser_accessibility_manager_win.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/content/browser/accessibility/browser_accessibility_manager_win.cc b/content/browser/accessibility/browser_accessibility_manager_win.cc index 31f9d8a..69c8891 100644 --- a/content/browser/accessibility/browser_accessibility_manager_win.cc +++ b/content/browser/accessibility/browser_accessibility_manager_win.cc @@ -230,7 +230,7 @@ void BrowserAccessibilityManagerWin::NotifyAccessibilityEvent( } void BrowserAccessibilityManagerWin::OnRootChanged(ui::AXNode* new_root) { - if (delegate_ && delegate_->HasFocus()) + if (delegate_ && delegate_->AccessibilityViewHasFocus()) NotifyAccessibilityEvent(ui::AX_EVENT_FOCUS, GetRoot()); } |