2009-03-04 Xan Lopez <xan@gnome.org>
authorxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Mar 2009 19:06:32 +0000 (19:06 +0000)
committerxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Mar 2009 19:06:32 +0000 (19:06 +0000)
        Reviewed by Holger Freyther.

        https://bugs.webkit.org/show_bug.cgi?id=24358
        [GTK] Scrollbars not clipped correctly

        Do not take into account the case of being a ScrollView scrollbar,
        since those are native in our case.

        * platform/gtk/ScrollbarGtk.cpp:
        (ScrollbarGtk::frameRectsChanged):

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

WebCore/ChangeLog
WebCore/platform/gtk/ScrollbarGtk.cpp

index 5b3a1159a422b8eb6c9dc11a7aef157b8117b09d..de48d437837130399792fd2ff54892dc3ddd22ea 100644 (file)
@@ -1,3 +1,16 @@
+2009-03-04  Xan Lopez  <xan@gnome.org>
+
+        Reviewed by Holger Freyther.
+
+        https://bugs.webkit.org/show_bug.cgi?id=24358
+        [GTK] Scrollbars not clipped correctly
+
+        Do not take into account the case of being a ScrollView scrollbar,
+        since those are native in our case.
+
+        * platform/gtk/ScrollbarGtk.cpp:
+        (ScrollbarGtk::frameRectsChanged):
+
 2009-03-04  Xan Lopez  <xan@gnome.org>
 
         Reviewed by Holger Freyther.
index 073250eab9862b3a2a503b4b557c848a3d4d14e8..be4c1c401e619f7753a9ac5baa06eea9e892f198 100644 (file)
@@ -70,19 +70,9 @@ void ScrollbarGtk::frameRectsChanged()
     if (!parent())
         return;
 
-    IntPoint loc;
-
-    /*
-     * The same scrollbars are used for ScrollViews and 'floating divs'/
-     * RenderLayout. We need to take this into account to decide which
-     * function to use to transform the location coordinates.
-     * The basic difference is that RenderLayout scrollbars need to have
-     * substracted the scrollOffset() from their location.
-     */
-    if (parent()->isScrollViewScrollbar(this))
-        loc = parent()->convertToContainingWindow(frameRect().location());
-    else
-        loc = parent()->contentsToWindow(frameRect().location());
+    // Translate our coordinates, we are a RenderLayout scrollbar because our
+    // ScrollView scrollbars are native.
+    IntPoint loc = parent()->contentsToWindow(frameRect().location());
 
     // Don't allow the allocation size to be negative
     IntSize sz = frameRect().size();