Bug 101046 – Imrpove scoring in touch adjustment to address bias towards smaller targets.

• Component: New Bugs
• Status: RESOLVED FIXED

[Kevin Ellis]

Imrpove scoring in touch adjustment to address bias towards smaller targets.

[Kevin Ellis]

Created attachment 172047 [details]
Patch

[Kevin Ellis]

The patch addresses problems with touch adjustment at the boundary of a flash plugin.  Previously, we normalized the overlap score with respect to the size of the candidate element.  The flash plugin being typically very large compared to the touch area resulted in a very poor overlap score even if the touch is entirely contained within the plugin boundary.

[Kevin Ellis]

Comment on attachment 172047 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=172047&action=review

> LayoutTests/touchadjustment/rotated-node.html:75
> +        testTouchPoint(touchPoint(71, 0, 20), e.container);

Original position clipped boundary due to rounding.  Bumping the offset slightly addresses the overlap, while still ensuring that the bounds overlaps the rectilinear bounds of the target.  Overlap of the rectilinear bounds is required to properly test that the candidate is rejected during touch adjustment.

[WebKit Review Bot]

Comment on attachment 172047 [details]
Patch

Clearing flags on attachment: 172047

Committed r133312: <http://trac.webkit.org/changeset/133312>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.