[Webkit-unassigned] [Bug 20013] Windows AX heuristics are poor

[bugzilla-daemon at webkit.org]

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


jhoneycutt at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22599|review?(jhoneycutt at apple.com|review-
               Flag|)                           |




------- Comment #64 from jhoneycutt at apple.com  2008-08-21 22:13 PDT -------
(From update of attachment 22599)
+        Add functions to navigate properly between the accessibility parent
+        and accessibility children. 

I'm not sure that this description gets the point across. The bug is that,
currently, calling previousSibling, nextSibling, etc on an accessibility object
could return items that shouldn't be exposed because their accessibility is
ignored, correct? This should be more explicit in the ChangeLog.

This patch contains the previous patch's changes to children/addChildren. These
should be removed from this patch.

I'm not sure that this patch will work properly if objects in the DOM are moved
between parents, or if objects are added or removed. This will need a layout
tests that confirm that, after these sorts of DOM operations, navigating
between elements gives the expected result. If you find that the behavior is
not correct, you might look at how parentObjectUnignored works and follow the
same method.

I think the name of the new methods should be consistent with the name of the
parentObjectUnignored method, so the methods should be renamed
nextSiblingUnignored, previousSiblingUnignored, and so on.

+    AccessibilityObject* parentObj = m_object->parentObjectUnignored();
+
+    if (parentObj)  {
+        *parent = static_cast<IDispatch*>(wrapper(parentObj));
+        (*parent)->AddRef();
+        return S_OK;
+    }
+
This is a separate bug that could have its own patch.

Thanks, Sankar! r- for these issues.


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