AX: VoiceOver not announcing the right header information for table on iOS

[Nan Wang]

1. Launch Safari and VoiceOver
2. Go to http://www.w3.org/WAI/tutorials/tables/caption-summary/
3. Swipe right until reach example of Approach 3
4. Listen to the announcement of each cell of this table

For the “Studio” cell, it’s announced as “1 bedroom, Studio”,
For the first column, “1 bedroom” is announced as “2 bedroom, 1 bedroom”.
It seemed those header informations were messed up by the empty cell at the beginning.

[Nan Wang]

<rdar://problem/22668467>

[Nan Wang]

Created attachment 274956 [details]
initial patch

My simulator keeps crashing. Will try to add a layout test later.

[chris fleizach]

Comment on attachment 274956 [details]
initial patch

View in context: https://bugs.webkit.org/attachment.cgi?id=274956&action=review

> Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:1067
> +        if (rowRangeIndex >= rowHeaderRange.first && rowRangeIndex <= rowHeaderRange.first + rowHeaderRange.second - 1) {

this should be equivalent right?

rowRangeIndex < rowHeaderRange.first + rowHeaderRange.second

> Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:1069
> +            if (wrapper)

if (AccessibilityObjectWrapper* wrapper = rowHeader->wrapper())
   [headers add...

[Nan Wang]

Created attachment 275158 [details]
patch

Got the simulator test working

[WebKit Commit Bot]

Comment on attachment 275158 [details]
patch

Clearing flags on attachment: 275158

Committed r198819: <http://trac.webkit.org/changeset/198819>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.