Summary: | Faulty logic in AccessibilityRenderObject::isSelected | ||
---|---|---|---|
Product: | WebKit | Reporter: | Xan Lopez <xan.lopez> |
Component: | Accessibility | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED WORKSFORME | ||
Severity: | Normal | CC: | cfleizach, jcraig, jhoneycutt, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | 528+ (Nightly build) | ||
Hardware: | PC | ||
OS: | All |
Description
Xan Lopez
2009-06-10 13:45:22 PDT
i think the last false is OK, since we don't want to return true for default for isSelected there's obviously something missing though that should return true for some state. This isSelected does not correspond to the selection (at least on the mac... that's represented by AXSelectedTextRange). it's used more in the context of a multi-select list where one of the list items is selected (i'm pretty sure that AccessibilityListBoxOption overrides this method) So while this code does not necessarily look correct, I'm not sure what elements WebKit does want to return true for isSelected... if we identify those elements and their states, we would want to change that isSelected() is not meant to be the TextSelection. It's for things like aria-selected=true |