[webkit-reviews] review granted: [Bug 133436] Don't scroll to non-integral scroll offsets : [Attachment 232351] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jun 1 14:48:40 PDT 2014

Darin Adler <darin at apple.com> has granted Simon Fraser (smfr)
<simon.fraser at apple.com>'s request for review:
Bug 133436: Don't scroll to non-integral scroll offsets

Attachment 232351: Patch

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=232351&action=review

>> Source/WebCore/page/scrolling/mac/ScrollingTreeFrameScrollingNodeMac.mm:325
>> +	FloatPoint roundedPosition(roundf(scrollPosition.x()),
> I am wondering whether the input devices that send integral values do integer
truncation or rounding. (and whether we should mimic that behavior)

Is this the single correct bottleneck for the rounding? What about the
setScrollPositionWithoutContentEdgeConstraints and setScrollLayerPosition

More information about the webkit-reviews mailing list