[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