2008-12-10 Simon Fraser <simon.fraser@apple.com>
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Dec 2008 01:55:06 +0000 (01:55 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 11 Dec 2008 01:55:06 +0000 (01:55 +0000)
        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

WebCore/ChangeLog
WebCore/rendering/RenderSlider.cpp

index 7b41d9c..387f4e5 100644 (file)
 
 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.
         
index 0d6a265..37ceac8 100644 (file)
@@ -251,7 +251,7 @@ void RenderSlider::updateFromElement()
         addChild(m_thumb->renderer());
     }
     setPositionFromValue();
-    setNeedsLayout(true);
+    setNeedsLayout(true, false);
 }
 
 bool RenderSlider::mouseEventIsInThumb(MouseEvent* evt)