Summary: | [ATK] atk_table_cell_get_position() should return values of aria-rowindex and aria-colindex, if present | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joanmarie Diggs <jdiggs> | ||||
Component: | Accessibility | Assignee: | Joanmarie Diggs <jdiggs> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aboxhall, akkaran046, apinheiro, buildbot, cfleizach, commit-queue, dmazzoni, jcraig, samuel_white, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Joanmarie Diggs
2017-04-22 16:46:24 PDT
Updating the summary because the object attributes are already done. Created attachment 321851 [details]
Patch
Comment on attachment 321851 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=321851&action=review > Source/WebCore/accessibility/atk/WebKitAccessibleInterfaceTableCell.cpp:126 > + rowIndex = columnRowRange.first; this should be columRowRange.second right? (In reply to chris fleizach from comment #4) > Comment on attachment 321851 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=321851&action=review > > > Source/WebCore/accessibility/atk/WebKitAccessibleInterfaceTableCell.cpp:126 > > + rowIndex = columnRowRange.first; > > this should be columRowRange.second right? The index/location is the range's first; the span/length is its second. Comment on attachment 321851 [details] Patch Clearing flags on attachment: 321851 Committed r222551: <http://trac.webkit.org/changeset/222551> All reviewed patches have been landed. Closing bug. |