RESOLVED FIXED 36121
[Qt] Make WebKit scrollbars work with tiling 
https://bugs.webkit.org/show_bug.cgi?id=36121
Summary [Qt] Make WebKit scrollbars work with tiling
Antti Koivisto
Reported Monday, March 15, 2010 5:05:50 PM UTC
Currently tiling only works correctly when the QGraphicsWebView is in resizesToContent mode. It should work without too, using the scrollbar overlay (already implemented for accelerated compositing).
Attachments
patch (11.72 KB, patch)
2010-03-15 09:16 PDT, Antti Koivisto 		no flags
DetailsFormatted DiffDiff
updated patch (11.77 KB, patch)
2010-03-16 05:28 PDT, Antti Koivisto 		hausmann: review+
DetailsFormatted DiffDiff
Screenshot (77.90 KB, image/png)
2010-03-16 08:57 PDT, Simon Hausmann 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 Monday, March 15, 2010 5:16:40 PM UTC
Created attachment 50713 [details] patch
Simon Hausmann
Comment 2 Monday, March 15, 2010 5:50:57 PM UTC
This works quite well now! The vertical scrollbar is broken for me though. Do you see that, too?
Kenneth Rohde Christiansen
Comment 3 Monday, March 15, 2010 10:21:54 PM UTC
Nice patch!
Simon Hausmann
Comment 4 Tuesday, March 16, 2010 9:39:45 AM UTC
Comment on attachment 50713 [details] patch > +void QGraphicsWebViewPrivate::createOrDeleteOverlay() > +{ > + bool useOverlay = false; > +#if USE(ACCELERATED_COMPOSITING) > + useOverlay = useOverlay || rootGraphicsLayer; > +#endif > +#if ENABLE(TILED_BACKING_STORE) > + useOverlay = useOverlay || QWebFramePrivate::core(q->page()->mainFrame())->tiledBackingStore(); Isn't another condition that we need the overlay only if we're _not_ in resizesToContents mode?
Antti Koivisto
Comment 5 Tuesday, March 16, 2010 1:09:00 PM UTC
(In reply to comment #2) > This works quite well now! The vertical scrollbar is broken for me though. Do > you see that, too? No, both scrollbars look and work fine (this is on mac). Anything special I need to do to see this?
Antti Koivisto
Comment 6 Tuesday, March 16, 2010 1:28:31 PM UTC
Created attachment 50784 [details] updated patch
Antti Koivisto
Comment 7 Tuesday, March 16, 2010 1:29:13 PM UTC
Comment on attachment 50784 [details] updated patch fixed the resizesToContents bug Simon noticed
Simon Hausmann
Comment 8 Tuesday, March 16, 2010 4:56:24 PM UTC
(In reply to comment #5) > (In reply to comment #2) > > This works quite well now! The vertical scrollbar is broken for me though. Do > > you see that, too? > > No, both scrollbars look and work fine (this is on mac). Anything special I > need to do to see this? Hmmm, I see it happening on Linux with the Oxygen style. For some reason it doesn't happen with the other styles.
Simon Hausmann
Comment 9 Tuesday, March 16, 2010 4:57:58 PM UTC
Created attachment 50797 [details] Screenshot Attached is a screenshot of the bug. It looks like it only affects the vertical scrollbar, and I think that could happen if the geometries passed to are weird.
Antti Koivisto
Comment 10 Tuesday, March 16, 2010 5:57:27 PM UTC
Sending WebCore/ChangeLog Sending WebCore/page/Frame.cpp Sending WebKit/qt/Api/qgraphicswebview.cpp Sending WebKit/qt/Api/qwebframe.cpp Sending WebKit/qt/Api/qwebframe_p.h Sending WebKit/qt/ChangeLog Transmitting file data ...... Committed revision 56068.
Note You need to log in before you can comment on or make changes to this bug.