If an accessibilityObject is focused, it calls cache->setIsSynchronizingSelection(true) assuming that the focus will cause a selection change, but doesn't validate that the change occurred or reset this state. After any event that's assuming a selection change will happen, m_isSynchronizingSelection should be reset to false.
Created attachment 255531 [details]
Comment on attachment 255531 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=255531&action=review
> + Bug 146300 AX: AccessibilityObject focus event that doesn't cause a selection
"focus events that don't cause a"
> + AXObjectCache* cache = renderObject.axObjectCache();
It looks like this method might be better off taking a point to the axobject cache
> + [rootWebArea()->wrapper() accessibilityPostedNotification:NSAccessibilityFocusedUIElementChangedNotification userInfo:nil];
since this looks like it's only for testing, i would change the name to something like "AXFocusedChanged" so that when checking this in the test it will be clear what the value should be, and
will also let other platforms follow suit
Created attachment 255597 [details]
I changed to AXFocusedChanged, but it's worth noting that other logic can still post NSAccessibilityFocusedUIElementChangedNotification so we may end up writing tests some times that need to listen for both. Probably not an issue, but thought I'd note it.
Comment on attachment 255597 [details]
Clearing flags on attachment: 255597
Committed r185974: <http://trac.webkit.org/changeset/185974>
All reviewed patches have been landed. Closing bug.