[Webkit-unassigned] [Bug 69440] AccessibilityMenuList does not fire change notification when popup is not open.

[bugzilla-daemon at webkit.org]

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


chris fleizach <cfleizach at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #109843|review?                     |review-
               Flag|                            |




--- Comment #2 from chris fleizach <cfleizach at apple.com>  2011-10-05 15:27:34 PST ---
(From update of attachment 109843)
View in context: https://bugs.webkit.org/attachment.cgi?id=109843&action=review

layout test needed. otherwise seems ok

> Source/WebCore/accessibility/AccessibilityMenuList.cpp:96
> +            popup->didUpdateActiveOption(optionIndex);

i think we should take this patch to verify (outside of ASSERT)  that childObjects[0].get() actually is a AccessibilityMenuListPopup, by doing something like if (childObjects[0]->isMenuListPopup())

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.