[webkit-reviews] review requested: [Bug 20277] implement window.scrollByPages and window.scrollByLines : [Attachment 23490] updated patch

[bugzilla-daemon at webkit.org]

arno. <arenevier at fdn.fr> has asked Darin Adler <darin at apple.com> for review:
Bug 20277: implement window.scrollByPages and window.scrollByLines
https://bugs.webkit.org/show_bug.cgi?id=20277

Attachment 23490: updated patch
https://bugs.webkit.org/attachment.cgi?id=23490&action=edit

------- Additional Comments from arno. <arenevier at fdn.fr>
Why are we doing updateLayout in scrollByLines? Is it that we are trying to
force the renderer be created?

Layout is not updated if I don't call updateLayout (ie: scrollbar does not
move, and window view does not change).

Here is a patch addressing your comments.