Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge r228417 - AX: defer focusedUIElement notifications
https://bugs.webkit.org/show_bug.cgi?id=182643 <rdar://problem/37394310> Reviewed by Zalan Bujtas. Source/WebCore: Deferring focus changes for accessibility has a number of benefits. 1) Reduces the chance of calling into layout during layout. 2) Coalesces multiple focus notifications that would be needlessly sent. 3) Improves performance by not calling out to the accessibility notification machinery during layout. In this patch, I also started making more AXObjectCache calls private. This will reduce the chance that clients will call into AXObjectCache during unexpected times. * accessibility/AXObjectCache.cpp: (WebCore::AXObjectCache::deferFocusedUIElementChangeIfNeeded): (WebCore::conditionallyAddNodeToFilterList): (WebCore::filterVectorPairForRemoval): (WebCore::filterMapForRemoval): (WebCore::filterListForRemoval): (WebCore::AXObjectCache::prepareForDocumentDestruction): (WebCore::AXObjectCache::performDeferredCacheUpdate): * accessibility/AXObjectCache.h: * dom/Document.cpp: (WebCore::Document::setFocusedElement): LayoutTests: * accessibility/mac/aria-menu-item-selected-notification.html: Rewrite test to accomodate that focus changes happen asynchronously. * accessibility/mac/selection-notification-focus-change-expected.txt: * platform/mac-wk2/accessibility/mac/selection-notification-focus-change-expected.txt: The order of notifications is different now that focus changes happen later.
- Loading branch information
1 parent
f8e9930
commit f8fab14
Showing
8 changed files
with
102 additions
and
31 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters