Reviewed by Darin Adler.
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Dec 2007 18:28:50 +0000 (18:28 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Dec 2007 18:28:50 +0000 (18:28 +0000)
        - fix <rdar://problem/5656368> REGRESSION(3.0.4-ToT): Acid2 test fails to render anything

        * platform/win/ScrollViewWin.cpp:
        (WebCore::ScrollView::maximumScroll): Changed back to return the maximum
        scroll offsets even if scrolling is not allowed, because navigation and
        scrollTo should still work.
        (WebCore::ScrollView::wheelEvent): Added early return if scrolling is
        not allowed.

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

WebCore/ChangeLog
WebCore/platform/win/ScrollViewWin.cpp

index 27cfef8232b6af5ad8b154e00152239dafd426a1..2026c06943250b37f2d1820cad353f363ac52443 100644 (file)
@@ -1,3 +1,16 @@
+2007-12-20  Dan Bernstein  <mitz@apple.com>
+
+        Reviewed by Darin Adler.
+
+        - fix <rdar://problem/5656368> REGRESSION(3.0.4-ToT): Acid2 test fails to render anything
+
+        * platform/win/ScrollViewWin.cpp:
+        (WebCore::ScrollView::maximumScroll): Changed back to return the maximum
+        scroll offsets even if scrolling is not allowed, because navigation and
+        scrollTo should still work.
+        (WebCore::ScrollView::wheelEvent): Added early return if scrolling is
+        not allowed.
+
 2007-12-17  Tony Chang  <idealisms@gmail.com>
 
         Reviewed by Darin.
index 0ed6431854ad2f8c0e2e7484244e6bb01682c85a..69853763cb685df17cbaca12925ca362bd6f5933 100644 (file)
@@ -347,8 +347,6 @@ IntSize ScrollView::scrollOffset() const
 
 IntSize ScrollView::maximumScroll() const
 {
-    if (!m_data->allowsScrolling())
-        return IntSize();
     IntSize delta = (m_data->m_contentsSize - IntSize(visibleWidth(), visibleHeight())) - scrollOffset();
     delta.clampNegativeToZero();
     return delta;
@@ -665,6 +663,9 @@ void ScrollView::themeChanged()
 
 void ScrollView::wheelEvent(PlatformWheelEvent& e)
 {
+    if (!m_data->allowsScrolling())
+        return;
+
     // Determine how much we want to scroll.  If we can move at all, we will accept the event.
     IntSize maxScrollDelta = maximumScroll();
     if ((e.deltaX() < 0 && maxScrollDelta.width() > 0) ||