[Webkit-unassigned] [Bug 40197] Enhance the hit testing to take a rectangle instead of a point

[bugzilla-daemon at webkit.org]

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


Grace Kloba <klobag at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #57979|                            |review?
               Flag|                            |




--- Comment #4 from Grace Kloba <klobag at gmail.com>  2010-06-06 12:37:53 PST ---
Created an attachment (id=57979)
 --> (https://bugs.webkit.org/attachment.cgi?id=57979)
first in-progress patch

This is an in-progress patch. The goal is to get the feedback from the broader audience.

The key change is EventHandler::hitTestResultAtPoint(). If the pointPadding is (0, 0), the functionality should be identical as before. If pointPadding is not empty, we will do a region test. Inside the returned HitTestResult, besides innerNode() you can also get a list of Node overlapped with the "fat" point by calling rawNodeList(). The Nodes are ordered based on z-order.

There are couple of open issues.
1. currently all the Nodes are in the same widget/frame as the innerNode(), do we need to support Nodes from different frames? (See code in EventHandler.cpp)

2. currently it doesn't extend isPointInOverflowControl() for the region test, is it needed? (See RenderBlock.cpp)

3. currently it doesn't support SVG. (See RenderSVGRoot.cpp)

4. currently there is no special treatment for transform3D as I don't know what to do. (See RenderLayer.cpp)

5. need suggestion on writing layout test

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