[Webkit-unassigned] [Bug 25534] [GTK] Objects of ROLE_TABLE should implement the accessible table interface
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Oct 26 14:27:22 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=25534
Joanmarie Diggs <joanmarie.diggs at gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #41642|0 |1
is obsolete| |
Attachment #41642|review? |
Flag| |
Attachment #41898| |review?
Flag| |
--- Comment #7 from Joanmarie Diggs <joanmarie.diggs at gmail.com> 2009-10-26 14:27:21 PDT ---
Created an attachment (id=41898)
--> (https://bugs.webkit.org/attachment.cgi?id=41898)
table 1 (breaking these apart for easier review)
It's rough being the new kid. ;-)
Okay. This is based on the original patch reviewed by Xan. Changes:
1. Makes the suggested (and then withdrawn) change to use std:find.
2. Correct the guint/gint error
3. Acknowledges, but makes no changes in response to, this comment:
> Just a sanity check: what comes in that pair, the indexes of the columns the
> cell occupies?
Nope. It's the index of the column and then the column span:
void AccessibilityTableCell::columnIndexRange(pair<int, int>& columnRange)
{
if (!m_renderer || !m_renderer->isTableCell())
return;
RenderTableCell* renderCell = toRenderTableCell(m_renderer);
columnRange.first = renderCell->col();
columnRange.second = renderCell->colSpan();
}
> If the function returns the extent shouldn't you return second -
> first?
I need the colSpan, columnRange.second contains the colSpan. Testing with
tables with multiple spans would also seem to indicate that what I have done is
sound.
4. Removes webkit_accessible_table_get_column_header. Xan suggests I'll figure
it out in a follow-up patch. Agreed. Might as well just introduce it into the
code then.
Please review. Thanks!
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list