[webkit-reviews] review granted: [Bug 109370] [chromium] Apply page scale to all WebInputEvent types : [Attachment 187736] Patch

[bugzilla-daemon at webkit.org]

James Robinson <jamesr at chromium.org> has granted Alexandre Elias
<aelias at chromium.org>'s request for review:
Bug 109370: [chromium] Apply page scale to all WebInputEvent types
https://bugs.webkit.org/show_bug.cgi?id=109370

Attachment 187736: Patch
https://bugs.webkit.org/attachment.cgi?id=187736&action=review

------- Additional Comments from James Robinson <jamesr at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=187736&action=review


R=me, one question for your consideration

> Source/WebKit/chromium/src/WebInputEventConversion.cpp:178
> +	   m_area = IntSize(e.data.tap.width / scale, e.data.tap.height /
scale);

do we want to use an explicit conversion here so we know what sort of rounding
we're getting?	Maybe go to a FloatSize first and then use
enclosing/enclosed/whateverIntSize?