[webkit-changes] [WebKit/WebKit] 45ff67: AX: display:contents elements are sometimes missin...

Tyler Wilcock noreply at github.com
Fri Jul 28 16:12:32 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 45ff67ff20ee13b227eede764c0a4e2f4ee8603f
      https://github.com/WebKit/WebKit/commit/45ff67ff20ee13b227eede764c0a4e2f4ee8603f
  Author: Tyler Wilcock <tyler_w at apple.com>
  Date:   2023-07-28 (Fri, 28 Jul 2023)

  Changed paths:
    A LayoutTests/accessibility/display-contents/table-dynamic-expected.txt
    A LayoutTests/accessibility/display-contents/table-dynamic.html
    A LayoutTests/platform/glib/accessibility/display-contents/table-dynamic-expected.txt
    A LayoutTests/platform/ios/accessibility/display-contents/table-dynamic-expected.txt
    M Source/WebCore/accessibility/AccessibilityObject.cpp

  Log Message:
  -----------
  AX: display:contents elements are sometimes missing their children
https://bugs.webkit.org/show_bug.cgi?id=259608
rdar://problem/113044333

Reviewed by Chris Fleizach.

This happened because AccessibilityObject::insertChild detects and bails when an object is trying to insert a child
that belongs to a display:contents object that is not `this`. But if the correct parent of that child does not have
it’s dirty-children bit set, we never actually insert the child, resulting in it being missing from the accessibility
tree. With this patch, we set that bit, ensuring the accessibility tree is updated correctly.

* LayoutTests/accessibility/display-contents/table-dynamic-expected.txt: Added.
* LayoutTests/accessibility/display-contents/table-dynamic.html: Added.
* LayoutTests/platform/ios/accessibility/display-contents/table-dynamic-expected.txt: Added.
* Source/WebCore/accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::insertChild):

Canonical link: https://commits.webkit.org/266407@main




More information about the webkit-changes mailing list