Bug 19367

Summary: Plugins are drawn over the scrollbars
Product: WebKit Reporter: Pierre-Luc Beaudoin <pierre-luc.beaudoin>
Component: WebKit QtAssignee: marcoil <marc.ordinasillopis>
Status: RESOLVED FIXED    
Severity: Normal CC: hausmann, siraj, tonikitoo
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
URL: http://www.radio-canada.ca/nouvelles/
Attachments:
Description Flags
Fixes the bug for r34498 none

Pierre-Luc Beaudoin
Reported 2008-06-02 20:54:34 PDT
On this page, when resizing the window so that scrollbars overlap the flash content or ads, the flash content gets painted over the scrollbars. Also verified on the Gtk port: not affected.
Attachments
Fixes the bug for r34498 (1.25 KB, patch)
2008-06-11 13:57 PDT, marcoil 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
marcoil
Comment 1 2008-06-11 13:57:34 PDT
Created attachment 21643 [details] Fixes the bug for r34498 This patch fixes the issue clipping the embedded plugin window.
Simon Hausmann
Comment 2 2008-06-13 05:52:59 PDT
Comment on attachment 21643 [details] Fixes the bug for r34498 Nice trick :), looks good to me.
Simon Hausmann
Comment 3 2008-06-13 05:53:17 PDT
Landed in r34519
marcoil
Comment 4 2008-06-13 08:19:13 PDT
Reopening the bug as it's still present when the plugin is a native QWidget. I think there's two ways we could go fixing this: 1) Move the clipping handling up to WidgetQt. In this case we could also get rid of PluginView::m_window, as it's just a copy of WidgetPrivate::m_widget. 2) Move the native QWidget plugin support down from WidgetQt to PluginView. Now that we have NPAPI plugin support we could also move it all to an external plugin. Opinions?
Mark Rowe (bdash)
Comment 5 2008-06-22 18:23:44 PDT
Comment on attachment 21643 [details] Fixes the bug for r34498 Clearing review since the patch was landed.
Simon Hausmann
Comment 6 2008-11-20 04:46:08 PST
I believe Tor Arne fixed this one in r38560
Note You need to log in before you can comment on or make changes to this bug.