[webkit-reviews] review granted: [Bug 96017] Move RenderView::setFixedPositionedObjectsNeedLayout to FrameView : [Attachment 162574] Patch

[bugzilla-daemon at webkit.org]

James Robinson <jamesr at chromium.org> has granted Simon Fraser (smfr)
<simon.fraser at apple.com>'s request for review:
Bug 96017: Move RenderView::setFixedPositionedObjectsNeedLayout to FrameView
https://bugs.webkit.org/show_bug.cgi?id=96017

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

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


> Source/WebCore/page/FrameView.h:-200
> -    // This includes position:fixed and sticky objects.

This comment was nice - I'm not sure if "viewport constrained" has quite caught
on enough for everyone to know that it includes position:fixed and
position:sticky