[Webkit-unassigned] [Bug 79080] Row and column headers not exposed by WebCore accessibility

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=79080





--- Comment #19 from Aaron Leventhal <aaronlevbugs at gmail.com>  2012-03-20 13:53:49 PST ---
Thanks Chris. I'm trying to keep changes minimal, and reuse the existing code. But I either have to reorganize the existing code a bit or we end up with extra iterations for every cell.

For example, I can implement
AccessibilityTableColumn::isHeaderObject()
and call it on initialization of a cell so that the role is properly set.
However, for AccessibilityTableColumn::isHeaderObject() it will end up iterating through the same <thead> or <tbody> cells for each item in the column.

What do you think?

It might be better to make each cell responsible for calculating and caching its role. Then AccessibilityTableColumn::headerObject() could be dumb, and simply iterate through the column and check the role until the ColumnHeaderRole is found, similar to what it does for ARIA tables. 

Advantages:
- Better performance
- It would be easy to later change it to support multiple row and column headers.

Disadvantage:
- Bigger change

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.