Bug 47096

Summary: [GTK] Interior scrollbars do not draw their background properly
Product: WebKit Reporter: Martin Robinson <mrobinson>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch for this issue none

Description Martin Robinson 2010-10-04 11:17:41 PDT
This only manifests in a very few GTK+ themes -- I've only been able to reproduce it so far with the default GTK+ theme. The issue here is that the code that draws the scrolled window background behind the scrollbar is using gtk_paint_box instead of gtk_paint_shadow. The latter is the correct behavior, which can be observed in gtkscrolledwindow.c in the GTK+ sources.
Comment 1 Martin Robinson 2010-10-04 11:26:00 PDT
Created attachment 69649 [details]
Patch for this issue
Comment 2 Xan Lopez 2010-10-04 20:31:03 PDT
Comment on attachment 69649 [details]
Patch for this issue

One day we'll get rid of GTK+ themes and the world will be a better place.
Comment 3 Martin Robinson 2010-10-04 20:51:19 PDT
Thanks for the review! For posterity, I should correct my previous statement to say that I can only reproduce this issue with the default Ubuntu 9.10 theme.
Comment 4 Martin Robinson 2010-10-05 10:03:21 PDT
Comment on attachment 69649 [details]
Patch for this issue

Clearing flags on attachment: 69649

Committed r69119: <http://trac.webkit.org/changeset/69119>
Comment 5 Martin Robinson 2010-10-05 10:03:25 PDT
All reviewed patches have been landed.  Closing bug.