1020510 - let tbody/etc accessible in table hierarchy

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

spun off the bug 1005271

we fail to create table hierarchy if tbody etc get an accessible what happens in scrollable tables

Comment 1

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 2

[Trevor Saunders (:tbsaunde)]

Comment on attachment 8435169 [details] [diff] [review]
patch

>+      Accessible* table = aContext->IsTable() ?
>+        aContext :
>+        (aContext->Parent()->IsTable() ? aContext->Parent() : nullptr);
>+
>+      if (table) {
>         nsIContent* parentContent = aContent->GetParent();
>         nsIFrame* parentFrame = parentContent->GetPrimaryFrame();
>-        if (parentFrame->GetType() == nsGkAtoms::tableRowGroupFrame) {
>+        if (parentFrame->GetType() != nsGkAtoms::tableOuterFrame) {
>           parentContent = parentContent->GetParent();
>           parentFrame = parentContent->GetPrimaryFrame();

I sort of wish we had only one check if the parent or grand parent  was the table, but I guess this is fine.

I'm a little worried something might rely on the child of a table being a table row, but I couldn't immediately find anything.

Comment 3

[alexander :surkov (:asurkov)]

(In reply to Trevor Saunders (:tbsaunde) from comment #2)

> I sort of wish we had only one check if the parent or grand parent  was the
> table, but I guess this is fine.

it's felt a bit complicated but just keeping closer to original code 

> I'm a little worried something might rely on the child of a table being a
> table row, but I couldn't immediately find anything.

I tried to add some mochitests worrying about same thing but run into bug 1020603.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/d4fc8141e2db