REGRESSION(r120832): RenderLayer::clampScrollOffset doesn't properly clamp
[WebKit.git] / Source / WebCore / ChangeLog
index d3de75d..614af0f 100644 (file)
@@ -1,3 +1,19 @@
+2012-09-04  Julien Chaffraix  <jchaffraix@webkit.org>
+
+        REGRESSION(r120832): RenderLayer::clampScrollOffset doesn't properly clamp
+        https://bugs.webkit.org/show_bug.cgi?id=95776
+
+        Reviewed by Simon Fraser.
+
+        r120832 consolidated the clamping logic into RenderLayer::clampScrollOffset. The existing code wouldn't properly ensure that
+        the offset were positive which got exposed to other code paths, leading to the regression.
+
+        Tested by ManualTests/select-menu-list-wrongly-positioned.html as I didn't find a way to create a reliable layout test.
+
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::clampScrollOffset):
+        Fixed the clamping logic to ensure that the scroll offset's dimensions are positive.
+
 2012-09-04  Joshua Bell  <jsbell@chromium.org>
 
         IndexedDB: IDBRequest leaks if IDBCursor closes and no further events fired