[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 #22593|review?(jhoneycutt at apple.com|review-
               Flag|)                           |




------- Comment #63 from jhoneycutt at apple.com  2008-08-21 21:52 PDT -------
(From update of attachment 22593)
Sankar, thanks for the patch!

This patch will be much easier to review if it is split up into smaller
patches, and these changes will require layout tests to ensure that the
behavior is correct now and will not regress in the future.

The layout tests should test, for example, that:

<i style="display: block">Test</i> is not ignored and is represented in the
tree as a container element, and

<i role="button">Test</i> is not ignored and is represented in the tree as a
push button.

I am not sure that testing objects by tag name is the right method for this,
because I think that Firefox 3 only ignores elements based on whether they are
inline.

This patch also contains your previous patch's changes to the
children/addChildren methods. Those should be removed.

+    virtual bool isTextFormattingElement() const;
+    virtual bool isLinkOrListItemElement() const;
+    virtual bool isFormOrTableElement() const;
I don't think any of these methods need to be virtual methods on
AccessibilityObject.

r-


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