Title: [262073] trunk/Source/WebCore
- Revision
- 262073
- Author
- [email protected]
- Date
- 2020-05-22 14:03:59 -0700 (Fri, 22 May 2020)
Log Message
Updates to the isolated tree must happen before posting notifications to clients.
https://bugs.webkit.org/show_bug.cgi?id=212266
Reviewed by Chris Fleizach.
Multiple tests.
In AXObjectCache::notificationPostTimerFired we were updating the
isolated tree after the notifications were posted to the platform
clients. This caused that in some cases when the client requested info
as the result of those notifications, the isolated tree was out-of-date.
In this patch updateIsolatedTree is called before notifying platform clients.
* accessibility/AXObjectCache.cpp:
(WebCore::AXObjectCache::notificationPostTimerFired):
(WebCore::AXObjectCache::postNotification):
(WebCore::AXObjectCache::postTextStateChangeNotification):
(WebCore::AXObjectCache::updateIsolatedTree):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (262072 => 262073)
--- trunk/Source/WebCore/ChangeLog 2020-05-22 20:52:28 UTC (rev 262072)
+++ trunk/Source/WebCore/ChangeLog 2020-05-22 21:03:59 UTC (rev 262073)
@@ -1,3 +1,24 @@
+2020-05-22 Andres Gonzalez <[email protected]>
+
+ Updates to the isolated tree must happen before posting notifications to clients.
+ https://bugs.webkit.org/show_bug.cgi?id=212266
+
+ Reviewed by Chris Fleizach.
+
+ Multiple tests.
+
+ In AXObjectCache::notificationPostTimerFired we were updating the
+ isolated tree after the notifications were posted to the platform
+ clients. This caused that in some cases when the client requested info
+ as the result of those notifications, the isolated tree was out-of-date.
+ In this patch updateIsolatedTree is called before notifying platform clients.
+
+ * accessibility/AXObjectCache.cpp:
+ (WebCore::AXObjectCache::notificationPostTimerFired):
+ (WebCore::AXObjectCache::postNotification):
+ (WebCore::AXObjectCache::postTextStateChangeNotification):
+ (WebCore::AXObjectCache::updateIsolatedTree):
+
2020-05-22 Sam Weinig <[email protected]>
Extended Color Cleanup: Make alpha premultiplication code more consistent and clear regarding what works with extended colors
Modified: trunk/Source/WebCore/accessibility/AXObjectCache.cpp (262072 => 262073)
--- trunk/Source/WebCore/accessibility/AXObjectCache.cpp 2020-05-22 20:52:28 UTC (rev 262072)
+++ trunk/Source/WebCore/accessibility/AXObjectCache.cpp 2020-05-22 21:03:59 UTC (rev 262073)
@@ -1013,6 +1013,7 @@
void AXObjectCache::notificationPostTimerFired()
{
+ AXTRACE("AXObjectCache::notificationPostTimerFired");
// During LayoutTests, accessibility may be disabled between the time the notifications are queued and the timer fires.
// Thus check here and return if accessibility is disabled.
if (!accessibilityEnabled())
@@ -1028,42 +1029,44 @@
// when the notification list is cleared at the end. Instead copy this list at the start.
auto notifications = WTFMove(m_notificationsToPost);
+ // Filter out the notifications that are not going to be posted to platform clients.
+ Vector<std::pair<RefPtr<AXCoreObject>, AXNotification>> notificationsToPost;
+ notificationsToPost.reserveCapacity(notifications.size());
for (const auto& note : notifications) {
- AXCoreObject* obj = note.first.get();
- if (!obj->objectID())
+ ASSERT(note.first);
+ if (!note.first->objectID() || !note.first->axObjectCache())
continue;
- if (!obj->axObjectCache())
- continue;
-
#ifndef NDEBUG
// Make sure none of the render views are in the process of being layed out.
// Notifications should only be sent after the renderer has finished
- if (is<AccessibilityRenderObject>(*obj)) {
- if (auto* renderer = downcast<AccessibilityRenderObject>(*obj).renderer())
+ if (is<AccessibilityRenderObject>(*note.first)) {
+ if (auto* renderer = downcast<AccessibilityRenderObject>(*note.first).renderer())
ASSERT(!renderer->view().frameView().layoutContext().layoutState());
}
#endif
- AXNotification notification = note.second;
-
// Ensure that this menu really is a menu. We do this check here so that we don't have to create
// the axChildren when the menu is marked as opening.
- if (notification == AXMenuOpened) {
- obj->updateChildrenIfNecessary();
- if (obj->roleValue() != AccessibilityRole::Menu)
+ if (note.second == AXMenuOpened) {
+ note.first->updateChildrenIfNecessary();
+ if (note.first->roleValue() != AccessibilityRole::Menu)
continue;
}
-
- if (notification == AXChildrenChanged && obj->parentObjectIfExists() && obj->lastKnownIsIgnoredValue() != obj->accessibilityIsIgnored())
- childrenChanged(obj->parentObject());
- postPlatformNotification(obj, notification);
+ if (note.second == AXChildrenChanged && note.first->parentObjectIfExists()
+ && note.first->lastKnownIsIgnoredValue() != note.first->accessibilityIsIgnored())
+ childrenChanged(note.first->parentObject());
+
+ notificationsToPost.append(note);
}
#if ENABLE(ACCESSIBILITY_ISOLATED_TREE)
- updateIsolatedTree(notifications);
+ updateIsolatedTree(notificationsToPost);
#endif
+
+ for (const auto& note : notificationsToPost)
+ postPlatformNotification(note.first.get(), note.second);
}
void AXObjectCache::passwordNotificationPostTimerFired()
@@ -1387,6 +1390,7 @@
void AXObjectCache::postTextStateChangeNotification(AccessibilityObject* object, const AXTextStateChangeIntent& intent, const VisibleSelection& selection)
{
+ AXTRACE("AXObjectCache::postTextStateChangeNotification");
stopCachingComputedObjectAttributes();
#if PLATFORM(COCOA)
@@ -1421,6 +1425,7 @@
void AXObjectCache::postTextStateChangeNotification(Node* node, AXTextEditType type, const String& text, const VisiblePosition& position)
{
+ AXTRACE("AXObjectCache::postTextStateChangeNotification");
if (!node || type == AXTextEditTypeUnknown)
return;
@@ -3138,7 +3143,7 @@
AXLOG(std::make_pair(&object, notification));
AXLOG(*this);
- if (!m_pageID)
+ if (!m_pageID || object.objectID() == InvalidAXID)
return;
auto tree = AXIsolatedTree::treeForPageID(*m_pageID);
@@ -3148,11 +3153,12 @@
switch (notification) {
case AXCheckedStateChanged:
case AXSelectedTextChanged:
- case AXValueChanged: {
- if (object.objectID() != InvalidAXID)
- tree->updateNode(object);
+ case AXValueChanged:
+ tree->updateNode(object);
break;
- }
+ case AXChildrenChanged:
+ tree->updateChildren(object);
+ break;
default:
break;
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
