[webkit-reviews] review granted: [Bug 60677] Layout unnecessarily interrupts smooth scrolling at msnbc.com : [Attachment 93220] Avoid an unnecessary immediate scroll in updateScrollbars()

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted mitz at webkit.org's request for review:
Bug 60677: Layout unnecessarily interrupts smooth scrolling at msnbc.com
https://bugs.webkit.org/show_bug.cgi?id=60677

Attachment 93220: Avoid an unnecessary immediate scroll in updateScrollbars()
https://bugs.webkit.org/attachment.cgi?id=93220&action=review

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

> Source/WebCore/platform/ScrollView.cpp:586
> +    IntPoint scrollPoint =
adjustScrollPositionWithinRange(IntPoint(desiredOffset)) +
IntSize(m_scrollOrigin.x(), m_scrollOrigin.y());

Wish there was a cleaner way to convert a point to a size.