Bug 21303 - [GTK] Scrolling glitches
Summary: [GTK] Scrolling glitches
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-02 03:57 PDT by Mikael Tennhammar
Modified: 2008-10-20 08:41 PDT (History)
2 users (show)

See Also:


Attachments
Changes in ScrollViewGtk to have teh view repainted (605 bytes, patch)
2008-10-02 03:59 PDT, Mikael Tennhammar
no flags Details | Formatted Diff | Diff
Fix the repainting of the content when scrolling via the scrollbars (1.09 KB, patch)
2008-10-03 03:45 PDT, Mikael Tennhammar
no flags Details | Formatted Diff | Diff
Fix the repainting of the content when scrolling via the scrollbars (587 bytes, patch)
2008-10-03 04:43 PDT, Mikael Tennhammar
hyatt: review-
Details | Formatted Diff | Diff
Kill off more of ScrollViewGtk, fix GTK+ rendering issues (25.81 KB, patch)
2008-10-03 05:24 PDT, Alp Toker
hyatt: review-
Details | Formatted Diff | Diff
Scrolling fixes (7.50 KB, patch)
2008-10-18 00:28 PDT, Alp Toker
hyatt: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mikael Tennhammar 2008-10-02 03:57:04 PDT
The visual area is not updated when auto scrolling.
Comment 1 Mikael Tennhammar 2008-10-02 03:59:22 PDT
Created attachment 24016 [details]
Changes in ScrollViewGtk to have teh view repainted
Comment 2 Dave Hyatt 2008-10-02 04:27:56 PDT
Comment on attachment 24016 [details]
Changes in ScrollViewGtk to have teh view repainted

r=me

Gtk had implemented the update() method incorrectly (when it was never meant to be an invalidate), so I guess this got caused by me fixing that.
Comment 3 Dave Hyatt 2008-10-02 11:24:46 PDT
I landed this with my next round of ScrollView changes in 37198.

Comment 4 Mikael Tennhammar 2008-10-03 03:45:20 PDT
Created attachment 24046 [details]
Fix the repainting of the content when scrolling via the scrollbars

The current code do not make that SccrollView repaint itself. after scroll.
Note For the reviewer! I have only experience from GTK, but it seems clean to call the ScrollView's own method.
Comment 5 Mikael Tennhammar 2008-10-03 03:47:15 PDT
After latest updates of ScrollViewGtk the repainting of the view does not happen.
Comment 6 Mikael Tennhammar 2008-10-03 04:43:08 PDT
Created attachment 24048 [details]
Fix the repainting of the content when scrolling via the scrollbars

The current code do not make that SccrollView repaint itself. after scroll.
Note For the reviewer! I have only experience from GTK, but it seems clean to
call the ScrollView's own method.

This is the same patch as the previous one, but there where some printout in the old one.
Comment 7 Alp Toker 2008-10-03 05:24:06 PDT
Created attachment 24049 [details]
Kill off more of ScrollViewGtk, fix GTK+ rendering issues

 WebCore/ChangeLog                             |   42 ++++++
 WebCore/platform/ScrollView.cpp               |   18 --
 WebCore/platform/ScrollView.h                 |    2 
 WebCore/platform/gtk/ScrollViewGtk.cpp        |  180 +++++++-------------------
 WebCore/platform/gtk/ScrollbarGtk.cpp         |  110 ++++++++++-----
 WebCore/platform/gtk/ScrollbarGtk.h           |    5 
 WebKit/gtk/ChangeLog                          |   28 ++++
 WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp |   50 ++++++-
 WebKit/gtk/webkit/webkitwebview.cpp           |   24 +++
 9 files changed, 263 insertions(+), 196 deletions(-)

So, I took the recent breakage as an opportunity to fix invalidation/repaint issues we've had in the GTK+ port since the early days and kill off the cruft in ScrollViewGtk/ScrollViewPrivate (which are almost dead now).
Comment 8 Dave Hyatt 2008-10-03 11:12:08 PDT
Comment on attachment 24048 [details]
Fix the repainting of the content when scrolling via the scrollbars

Made obsolete by my fixes last night.
Comment 9 Dave Hyatt 2008-10-03 11:12:58 PDT
Comment on attachment 24049 [details]
Kill off more of ScrollViewGtk, fix GTK+ rendering issues

You need to merge this patch to ToT.  I landed a fix for my Gtk autoscroll breakage last night.  I think this patch is better than what I did... you just need to get merged to ToT to get a new patch for review.
Comment 10 Dave Hyatt 2008-10-03 11:15:06 PDT
ScrollbarGtk(ScrollbarClient*, ScrollbarOrientation, ScrollbarControlSize, GtkAdjustment* = 0);

ScrollbarGtk is refcounted, so please make a create method instead and don't make your new constructor public.

Comment 11 Darin Adler 2008-10-12 19:21:22 PDT
Comment on attachment 24016 [details]
Changes in ScrollViewGtk to have teh view repainted

Clearing review flag from this obsolete patch.
Comment 12 Alp Toker 2008-10-18 00:28:14 PDT
Created attachment 24484 [details]
Scrolling fixes
Comment 13 Dave Hyatt 2008-10-20 08:26:53 PDT
Comment on attachment 24484 [details]
Scrolling fixes

r=me
Comment 14 Alp Toker 2008-10-20 08:41:50 PDT
Landed in r37725. (Would still be nice to pick up the ScrollbarGtk cleanup changes some time later, they weren't in the patch.)