Bug 23711 - [GTK] Assorted Scrollbar fixes
Summary: [GTK] Assorted Scrollbar fixes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-03 13:38 PST by Xan Lopez
Modified: 2009-02-03 16:41 PST (History)
0 users

See Also:


Attachments
Properly calculate scrollbar location (2.30 KB, patch)
2009-02-03 13:50 PST, Xan Lopez
zecke: review+
Details | Formatted Diff | Diff
Avoid needless castings. (3.74 KB, patch)
2009-02-03 13:51 PST, Xan Lopez
zecke: review+
Details | Formatted Diff | Diff
Do not hold extra ref to scrollbar (2.62 KB, patch)
2009-02-03 13:51 PST, Xan Lopez
zecke: review+
Details | Formatted Diff | Diff
Add manual test for floating div with scrollbar. (3.08 KB, patch)
2009-02-03 13:52 PST, Xan Lopez
zecke: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Xan Lopez 2009-02-03 13:38:36 PST
Fix invisible scrollbars in flotaing divs + a bunch of cleanups.
Comment 1 Xan Lopez 2009-02-03 13:50:31 PST
Created attachment 27288 [details]
Properly calculate scrollbar location
Comment 2 Xan Lopez 2009-02-03 13:51:05 PST
Created attachment 27289 [details]
Avoid needless castings.
Comment 3 Xan Lopez 2009-02-03 13:51:35 PST
Created attachment 27290 [details]
Do not hold extra ref to scrollbar
Comment 4 Xan Lopez 2009-02-03 13:52:01 PST
Created attachment 27291 [details]
Add manual test for floating div with scrollbar.
Comment 5 Holger Freyther 2009-02-03 14:21:09 PST
Comment on attachment 27290 [details]
Do not hold extra ref to scrollbar

We can kill the d'tor now. I will do it when landing the patch.
Comment 6 Holger Freyther 2009-02-03 16:41:15 PST
Landed in r40548 - r40551.