Bug 66180

Summary: AccessibilityObject levels are inconsistent
Product: WebKit Reporter: Samuel White <samuel.white>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cfleizach, samuel.white, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Level patch. none

Description Samuel White 2011-08-12 18:03:02 PDT
Currently, three types of AccessibilityObjects return a level value. That is, AccessibilityObjects that represent headings, blockquotes, or tables can have a level associated with them. At the moment, both the headingLevel and blockquoteLevel functions return 0 only when called on an object of the wrong type which makes 1 the smallest valid level each supports. However, tableLevel returns a value of 0 both when called on non-table AccessibilityObjects AND when called on a table that is not inside any other table.

Because some of these level values are not part of any W3C HTML draft and are currently only used by various screen readers I think WebKit should enforce some consistency via additional layout tests so this problem does not grow. This means that tableLevel should be fixed so that it reports a value of 0 only when it is called on a non-table object. Also, this would ensure that these level values remain consistant moving forward and if other levels become useful a central layout test would be available to ensure they too remain consistant.
Comment 1 Samuel White 2011-08-17 08:15:07 PDT
Created attachment 104174 [details]
Level patch.

This patch makes the level values returned by AccessibilityObjects consistent and adds a layout test to ensure that they stay that way moving forward.
Comment 2 chris fleizach 2011-08-17 09:22:37 PDT
Comment on attachment 104174 [details]
Level patch.

r=me
Comment 3 WebKit Review Bot 2011-08-17 11:28:15 PDT
Comment on attachment 104174 [details]
Level patch.

Clearing flags on attachment: 104174

Committed r93226: <http://trac.webkit.org/changeset/93226>
Comment 4 WebKit Review Bot 2011-08-17 11:28:20 PDT
All reviewed patches have been landed.  Closing bug.