Bug 27943

Summary: Need to implement ARIA role="radiogroup"
Product: WebKit Reporter: chris fleizach <cfleizach>
Component: AccessibilityAssignee: chris fleizach <cfleizach>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
patch darin: review+

Description chris fleizach 2009-08-03 09:41:28 PDT 
The ARIA spec defines the radiogroup role as "A group of radio controls." It will probably map to RadioGroupRole.
Comment 1 chris fleizach 2009-08-03 09:52:28 PDT 
Created attachment 33984 [details]
patch
Comment 2 Darin Adler 2009-08-03 10:00:15 PDT 
Comment on attachment 33984 [details]
patch

> +            AccessibilityObject* radioButton = m_object->selectedRadioButton();
> +            if (radioButton)
> +                return radioButton->wrapper();
> +            return nil;

We normally do the early exit for the failure case rather than the success case.

r=me
Comment 3 chris fleizach 2009-08-03 10:14:04 PDT 
http://trac.webkit.org/changeset/46721
Comment 4 chris fleizach 2009-08-03 12:39:22 PDT 
not sure why a layout test in mozilla/tables is failing for intel release after this patch.