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

Unified Diff: content/browser/accessibility/browser_accessibility_manager_win.cc

Issue 1899823002: Uses the activedescendant_changed event received from Blink to fire the right focus event (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed menu-list-open Layout test. Created 4 years, 7 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
Index: content/browser/accessibility/browser_accessibility_manager_win.cc
diff --git a/content/browser/accessibility/browser_accessibility_manager_win.cc b/content/browser/accessibility/browser_accessibility_manager_win.cc
index f09eea99cd42b1f9ae3ecdb3b0f1d2dcbe8e5c48..f5bf163e6e2b0a734cf4e7ef0b349c9b170bdae7 100644
--- a/content/browser/accessibility/browser_accessibility_manager_win.cc
+++ b/content/browser/accessibility/browser_accessibility_manager_win.cc
@@ -104,6 +104,11 @@ void BrowserAccessibilityManagerWin::UserIsReloading() {
}
}
+BrowserAccessibility* BrowserAccessibilityManagerWin::GetFocus() {
+ BrowserAccessibility* focus = BrowserAccessibilityManager::GetFocus();
+ return GetActiveDescendant(focus);
+}
+
void BrowserAccessibilityManagerWin::NotifyAccessibilityEvent(
BrowserAccessibilityEvent::Source source,
ui::AXEvent event_type,
@@ -231,6 +236,7 @@ bool BrowserAccessibilityManagerWin::CanFireEvents() {
void BrowserAccessibilityManagerWin::FireFocusEvent(
BrowserAccessibilityEvent::Source source,
BrowserAccessibility* node) {
+ DCHECK(node);
// On Windows, we always fire a FOCUS event on the root of a frame before
// firing a focus event within that frame.
if (node->manager() != last_focused_manager_ &&

Powered by Google App Engine
This is Rietveld 408576698