[Webkit-unassigned] [Bug 44472] Add AX notification for childrenChanged

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Aug 28 17:17:50 PDT 2010


https://bugs.webkit.org/show_bug.cgi?id=44472





--- Comment #20 from Daniel Bates <dbates at webkit.org>  2010-08-28 17:17:50 PST ---
(From update of attachment 65807)
>  void AXObjectCache::postPlatformNotification(AccessibilityObject* obj, AXNotification notification)
>  {
> -    if (notification != AXCheckedStateChanged)
> -        return;
> -
>      if (!obj || !obj->document() || !obj->documentFrameView())
>          return;
>  
>      ChromeClientChromium* client = toChromeClientChromium(obj->documentFrameView());
> -    if (client)
> -        client->didChangeAccessibilityObjectState(obj);
> +    if (client) {
> +        switch (notification) {
> +        case AXCheckedStateChanged:
> +            client->didChangeAccessibilityObjectState(obj);
> +            break;
> +        case AXChildrenChanged:
> +            client->didChangeAccessibilityObjectChildren(obj);
> +            break;
> +        }
> +    }

This switch statement caused compile-time failures on the Chromium Mac Release bot because it neither has a default case nor handles all the enum values of AXNotification.

-- 
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