Table headers are not retrieved properly by Accessibility API

Table headers are not retrieved properly by Accessibility API

Accessibility API for retrieving table headers do not work correctly.
It returns wrong number of headers and it's not suitable to the current logic that
determines table headers.

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

[k.czech, 2015-01-20 16:42:02]

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

[dmazzoni, 2015-01-21 07:12:30]

lgtm

https://codereview.chromium.org/862543003/diff/1/Source/modules/accessibility...
File Source/modules/accessibility/AXTableColumn.h (right):

https://codereview.chromium.org/862543003/diff/1/Source/modules/accessibility...
Source/modules/accessibility/AXTableColumn.h:51: // retrieves the "column"
headers (th, scope) from top to bottom for the each column.
nit: "for the each column" is confusing and can be deleted - since this is a
table column object it's obvious it applies to the column

[k.czech, 2015-01-21 13:50:16]

On 2015/01/21 07:12:30, dmazzoni wrote:
> lgtm
> 
>
https://codereview.chromium.org/862543003/diff/1/Source/modules/accessibility...
> File Source/modules/accessibility/AXTableColumn.h (right):
> 
>
https://codereview.chromium.org/862543003/diff/1/Source/modules/accessibility...
> Source/modules/accessibility/AXTableColumn.h:51: // retrieves the "column"
> headers (th, scope) from top to bottom for the each column.
> nit: "for the each column" is confusing and can be deleted - since this is a
> table column object it's obvious it applies to the column

Thanks Dominic

[k.czech, 2015-01-21 13:50:57]

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

[commit-bot: I haz the power, 2015-01-21 13:51:18]

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

[commit-bot: I haz the power, 2015-01-21 15:04:01]

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