2006-04-03 Justin Haygood <jhaygood@spsu.edu>
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Apr 2006 11:54:46 +0000 (11:54 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 4 Apr 2006 11:54:46 +0000 (11:54 +0000)
        Reviewed by eseidel.  Landed by eseidel.

        - WIN32: maximumScroll() is the maximum scroll delta, not the maximum scroll position. Update to use
          the real maximum scroll position.
        http://bugzilla.opendarwin.org/show_bug.cgi?id=8160

        * platform/win/ScrollViewWin.cpp:
        (WebCore::ScrollView::updateScrollBars):

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

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

index d1a94364f23642bd5369d7da3163181e0c0681c1..1062feaaa3fcea696b83f37d06dde29ee4be7fe0 100644 (file)
@@ -1,3 +1,14 @@
+2006-04-03  Justin Haygood <jhaygood@spsu.edu>
+
+        Reviewed by eseidel.  Landed by eseidel.
+        
+        - WIN32: maximumScroll() is the maximum scroll delta, not the maximum scroll position. Update to use
+          the real maximum scroll position.
+        http://bugzilla.opendarwin.org/show_bug.cgi?id=8160
+
+        * platform/win/ScrollViewWin.cpp:
+        (WebCore::ScrollView::updateScrollBars):
+
 2006-04-04  Eric Seidel  <eseidel@apple.com>
 
         Reviewed by andersca.
index a6b757bd63e0c26de8e0e6d4a53841b86daa0141..65c72b63c1f4ef42122aeb3bdf66195c18054ca4 100644 (file)
@@ -1,5 +1,6 @@
 /*
  * Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
+ * Copyright (C) 2006 Justin Haygood <jhaygood@spsu.edu>.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -246,7 +247,8 @@ int ScrollView::updateScrollInfo(short type, int current, int max, int pageSize)
 
 void ScrollView::updateScrollBars()
 { 
-    IntSize scroll = scrollOffset().shrunkTo(maximumScroll());
+    IntSize maxScrollPosition(contentsWidth(), contentsHeight());
+    IntSize scroll = scrollOffset().shrunkTo(maxScrollPosition);
     scroll.clampNegativeToZero();
 
     m_data->scrollOffset =