[Webkit-unassigned] [Bug 26856] AnchorElement, ButtonElement, InputButton and Document should fire focus event when it is clicked.

[bugzilla-daemon at webkit.org]

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


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #42159|review?                     |review+
               Flag|                            |




--- Comment #32 from Eric Seidel <eric at webkit.org>  2009-10-29 16:48:31 PDT ---
(From update of attachment 42159)
It's a bit strange to see this in the results:
 Anchor with tab index
 6 
 7 Anchor without tab index
 8 
 9 Link without tab index
 10 
 11 Link with tab index
 12 
 13 Link with contentEditable
 14 
 15 Link inside contentEditable

it's just noise.  For bonus points you could wrap it in a div and hwne running
under DRT remove it at the end of the test.

 74         // Make sure that links with tabIndex or contentEditable are still
focusable.

Maybe something more like: // Only allow links with a tabIndex or
contentEditable to be mouse-focusuable.

the "make sure" comment only makes sense given the previous code. :)

In general this looks fine though.

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