Widget's position change should not initiate layout, only when its size changes.
authorzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Nov 2013 07:31:52 +0000 (07:31 +0000)
committerzalan@apple.com <zalan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 6 Nov 2013 07:31:52 +0000 (07:31 +0000)
commit9fcd90cb99fbf36c7fbbeeced4034251b1709406
tree18e875b3f241bd6b1a67976e24988ba2da347e6f
parent96bec4c81eda88b53c67c21d50413c6763026165
Widget's position change should not initiate layout, only when its size changes.
https://bugs.webkit.org/show_bug.cgi?id=123860

Reviewed by Andreas Kling.

RenderWidgets initiate unnecessary layouts while scrolling when they are embedded to
overflow:scroll containers. Scroll position change doesn't dirty the render tree
so it should not trigger layout either.

.:

* ManualTests/layouts-on-renderwidgets-while-scrolling.html: Added.

Source/WebCore:

Manual test added. Unfortunately we can't test against the number of layouts yet.

* rendering/RenderWidget.cpp:
(WebCore::RenderWidget::setWidgetGeometry):
(WebCore::RenderWidget::updateWidgetGeometry):
(WebCore::RenderWidget::updateWidgetPosition):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@158727 268f45cc-cd09-0410-ab3c-d52691b4dbfc
ChangeLog
ManualTests/layouts-on-renderwidgets-while-scrolling.html [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderWidget.cpp