Bug 78262 - Add roundedIntPoint method for LayoutPoints
Summary: Add roundedIntPoint method for LayoutPoints
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Levi Weintraub
URL:
Keywords:
Depends on:
Blocks: 60318
  Show dependency treegraph
 
Reported: 2012-02-09 11:58 PST by Levi Weintraub
Modified: 2012-02-09 15:09 PST (History)
2 users (show)

See Also:


Attachments
Patch (17.42 KB, patch)
2012-02-09 13:40 PST, Levi Weintraub
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Levi Weintraub 2012-02-09 11:58:03 PST
Add in a stub roundedIntPoint method that operates on a LayoutPoint and returns a rounded IntPoint, and pipe it through its usage.
Comment 1 Levi Weintraub 2012-02-09 13:40:01 PST
Created attachment 126363 [details]
Patch
Comment 2 Eric Seidel (no email) 2012-02-09 14:58:12 PST
Comment on attachment 126363 [details]
Patch

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

These seem OK.  It's difficult for me to validate if they're correct.

> Source/WebCore/page/EventHandler.cpp:1049
> +                IntPoint mainFramePoint = mainView->rootViewToContents(resultView->contentsToRootView(roundedIntPoint(result.point())));

I guess our policy with hit testing is to round?
Comment 3 Emil A Eklund 2012-02-09 14:59:37 PST
(In reply to comment #2)
> I guess our policy with hit testing is to round?

At least for now. Making hit testing subpixel aware is something we want to do eventually.
Comment 4 Levi Weintraub 2012-02-09 15:09:42 PST
Comment on attachment 126363 [details]
Patch

Clearing flags on attachment: 126363

Committed r107296: <http://trac.webkit.org/changeset/107296>
Comment 5 Levi Weintraub 2012-02-09 15:09:45 PST
All reviewed patches have been landed.  Closing bug.