Aria table row and column headers are not exposed in accessibility

Aria table row and column headers are not exposed in accessibility

Aria table row and column headers should also be exposed
in accessibility as in html tables.

BUG=451047
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189033

[k.czech, 2015-01-23 15:19:38]

k.czech@samsung.com changed reviewers:
+ dmazzoni@chromium.org

[k.czech, 2015-01-23 15:19:39]

It should go in after https://codereview.chromium.org/807173005/.
It seems this patch may touch some browser's content tests.

[k.czech, 2015-01-23 15:21:22]

On 2015/01/23 15:19:39, k.czech wrote:
> It should go in after https://codereview.chromium.org/807173005/.
> It seems this patch may touch some browser's content tests.

Sorry, it should go in after this one https://codereview.chromium.org/873593003/

[dmazzoni, 2015-01-23 19:02:06]

lgtm

https://codereview.chromium.org/869253002/diff/1/LayoutTests/accessibility/ar...
File LayoutTests/accessibility/aria-tables.html (right):

https://codereview.chromium.org/869253002/diff/1/LayoutTests/accessibility/ar...
LayoutTests/accessibility/aria-tables.html:25: function
buildAccessibilityTree(accessibilityObject, i) {
We're using buildAccessibilityTree in 9 tests now - in a follow-up change could
you look into creating a test helper file that we could reuse in all of them?

If 1 or 2 of the tests are really different, that's fine - but I think many of
them are quite similar.

[k.czech, 2015-01-24 21:49:13]

On 2015/01/23 19:02:06, dmazzoni wrote:
> lgtm
> 
>
https://codereview.chromium.org/869253002/diff/1/LayoutTests/accessibility/ar...
> File LayoutTests/accessibility/aria-tables.html (right):
> 
>
https://codereview.chromium.org/869253002/diff/1/LayoutTests/accessibility/ar...
> LayoutTests/accessibility/aria-tables.html:25: function
> buildAccessibilityTree(accessibilityObject, i) {
> We're using buildAccessibilityTree in 9 tests now - in a follow-up change
could
> you look into creating a test helper file that we could reuse in all of them?
> 
> If 1 or 2 of the tests are really different, that's fine - but I think many of
> them are quite similar.

Yes, I will look into creating such a test helper file. It sounds as a good
idea.

[k.czech, 2015-01-27 13:14:05]

The CQ bit was checked by k.czech@samsung.com

[commit-bot: I haz the power, 2015-01-27 13:14:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/869253002/1

[commit-bot: I haz the power, 2015-01-27 14:26:59]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=189033