2008-12-10 Simon Fraser <simon.fraser@apple.com>

        Reviewed by Antti Koivisto

        <rdar://problem/6431224>

        When updating the value of a slider, don't mark the parents
        as needing layout, because the size of the slider can never
        change. This fixes full-page repaints in some cases.

        * rendering/RenderSlider.cpp:
        (WebCore::RenderSlider::updateFromElement):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@39192 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 7b41d9c..387f4e5 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -59,6 +59,19 @@
 
 2008-12-10  Simon Fraser  <simon.fraser@apple.com>
 
+        Reviewed by Antti Koivisto
+
+        <rdar://problem/6431224>
+        
+        When updating the value of a slider, don't mark the parents
+        as needing layout, because the size of the slider can never
+        change. This fixes full-page repaints in some cases.
+
+        * rendering/RenderSlider.cpp:
+        (WebCore::RenderSlider::updateFromElement):
+
+2008-12-10  Simon Fraser  <simon.fraser@apple.com>
+
         Potential build fix. The forward declaration of FloatPoint should
         be inside the WebCore namespace.
         

diff --git a/WebCore/rendering/RenderSlider.cpp b/WebCore/rendering/RenderSlider.cpp
index 0d6a265..37ceac8 100644 (file)
--- a/WebCore/rendering/RenderSlider.cpp
+++ b/WebCore/rendering/RenderSlider.cpp
@@ -251,7 +251,7 @@ void RenderSlider::updateFromElement()
         addChild(m_thumb->renderer());
     }
     setPositionFromValue();
-    setNeedsLayout(true);
+    setNeedsLayout(true, false);
 }
 
 bool RenderSlider::mouseEventIsInThumb(MouseEvent* evt)