User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.7) Gecko/20060921 Ubuntu/dapper-security Firefox/1.5.0.7 Build Identifier: version 3 alpha 1 (20061009) Accessibility information about the message-header tree table is incorrect: 1. The tree-table reports that it only has three columns 2. In the first row, only the information starting from the "Date" column is available. This problem means that assistive technologies like screen readers and Braille cannot present correct information about message headers. In other words, the message-header tree-table is inaccessible to people with visual disabilities. To duplicate this but, . and find the message-header table. Reproducible: Always Steps to Reproduce: 1. Run at-poke with Thunderbird version 3 2. Expand the at-poke information for Thunderbird 3. Examine the tree-table information for the message headers You can also run the Orca screen reader with Thunderbird version 3. Navigate to the message headers. As you down-arrow through the headers, Orca does not speak the message subject or sender. It will only speak the date and the two columns after the date. Actual Results: Orca does not speak the message subject or sender. It will only speak the date and the two columns after the date. Expected Results: Orca should speak all of the message-header information

The priority of this bug should be P2 and the severity major

confirmed. very odd, why the first mail starts from the Date, but mails below are fine. I also don't know why we only get 3 header columns

The cause of the bug: in nsXULTreeAccessible.cpp, GetFirstChild() calls GetCachedTreeitemAccessible() with a null column parameter, which makes GetCachedTreeitemAccessible() get the column from GetKeyColumn(). That's why the cells of the first row and before the sorted column are disappeared. The patch fixed the invoking of GetCachedTreeitemAccessible().

Comment on attachment 244865 [details] [diff] [review] patch Good. Please add 2 helper functions, GetFirstColumn, GetLastColumn.

addressing Ginn's comments.

Comment on attachment 244878 [details] [diff] [review] patch v2 I didn't look very close, but it looks right.

Checking in accessible/src/xul/nsXULTreeAccessible.cpp; /cvsroot/mozilla/accessible/src/xul/nsXULTreeAccessible.cpp,v <-- nsXULTreeAccessible.cpp new revision: 1.41; previous revision: 1.40 done