An invariant of our accessibility tree is that it doesn't include things that are ignored. Most paths end up at AccessibilityObject::insertChild, which prevents ignored children from being inserted. However, AccessibilityMenuListPopup::addChildren directly calls m_children.append, so it should ensure the things it adds aren't ignored first.
<rdar://problem/84062657>
Created attachment 440711 [details] Patch
Comment on attachment 440711 [details] Patch Is it possible to add an example of an ignored menu list item in a layout test?
(In reply to chris fleizach from comment #3) > Comment on attachment 440711 [details] > Patch > > Is it possible to add an example of an ignored menu list item in a layout > test? We can't test this because AccessibilityMenuListPopup is ignored on Mac and iOS, so its addChildren method never gets called, and thus we can't exercise the changed code. This patch would make this class do the right thing should it become un-ignored.
Which platforms will this have an effect on?
(In reply to Tyler Wilcock from comment #4) > (In reply to chris fleizach from comment #3) > > Comment on attachment 440711 [details] > > Patch > > > > Is it possible to add an example of an ignored menu list item in a layout > > test? > > We can't test this because AccessibilityMenuListPopup is ignored on Mac and > iOS, so its addChildren method never gets called, and thus we can't exercise > the changed code. This patch would make this class do the right thing should > it become un-ignored. How can AccessibilityMenuListPopup become un-ignored?