[webkit-reviews] review granted: [Bug 88475] Optimize FrameView::scrollXForFixedPosition() / scrollYForFixedPosition() : [Attachment 146160] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 6 18:14:05 PDT 2012


Sam Weinig <sam at webkit.org> has granted Simon Fraser (smfr)
<simon.fraser at apple.com>'s request for review:
Bug 88475: Optimize FrameView::scrollXForFixedPosition() /
scrollYForFixedPosition()
https://bugs.webkit.org/show_bug.cgi?id=88475

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

------- Additional Comments from Sam Weinig <sam at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=146160&action=review


> Source/WebCore/rendering/RenderLayer.h:114
> +    void move(LayoutSize size) { m_rect.move(size); }

This should probably be passed as a const& ref.


More information about the webkit-reviews mailing list