<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - AX: CSS table display styles can cause malformed, inaccessible AXTables to be exposed to the AX tree"
href="https://bugs.webkit.org/show_bug.cgi?id=136415#c16">Comment # 16</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - AX: CSS table display styles can cause malformed, inaccessible AXTables to be exposed to the AX tree"
href="https://bugs.webkit.org/show_bug.cgi?id=136415">bug 136415</a>
from <span class="vcard"><a class="email" href="mailto:n_wang@apple.com" title="Nan Wang <n_wang@apple.com>"> <span class="fn">Nan Wang</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=258382&action=diff" name="attach_258382" title="patch">attachment 258382</a> <a href="attachment.cgi?id=258382&action=edit" title="patch">[details]</a></span>
patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=258382&action=review">https://bugs.webkit.org/attachment.cgi?id=258382&action=review</a>
<span class="quote">>> Source/WebCore/accessibility/AccessibilityARIAGrid.cpp:91
>> + if (!rowChild->isTableRow() || (rowChild->renderer() && rowChild->renderer()->isAnonymous())) {
>
> shouldn't this check be in isTableRow()</span >
I tested with a case that table with CSS and no ARIA, and that made the table totally inaccessible by returning no rows. I think in AccessibilityTable::addChildren(), is<AccessibilityTableRow> is using the isTableRow() check. And the addChildren() logic is different from AccessibilityARIAGrid::addChildren(), so I'm little hesitate to break that logic.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>