RESOLVED FIXED 30214
[Qt] Windowless Plugins : Create Pixmap only when size changes. 
https://bugs.webkit.org/show_bug.cgi?id=30214
Summary [Qt] Windowless Plugins : Create Pixmap only when size changes.
Girish Ramakrishnan
Reported 2009-10-08 09:02:10 PDT
Currently, a pixmap is created when the cliprect/windowrect changes.
Attachments
Create X Pixmap only when size changes (1.73 KB, patch)
2009-10-08 09:03 PDT, Girish Ramakrishnan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Girish Ramakrishnan
Comment 1 2009-10-08 09:03:48 PDT
Created attachment 40876 [details] Create X Pixmap only when size changes
Eric Seidel (no email)
Comment 2 2009-10-08 09:36:46 PDT
Comment on attachment 40876 [details] Create X Pixmap only when size changes Can plugins become windowed from being non-windowed? The old code used to free m_drawable unconditionally, the new code only does so if the plugin is currently non-windowed.
Girish Ramakrishnan
Comment 3 2009-10-08 18:07:25 PDT
(In reply to comment #2) > (From update of attachment 40876 [details]) > Can plugins become windowed from being non-windowed? The old code used to free > m_drawable unconditionally, the new code only does so if the plugin is > currently non-windowed. Good point. But no, they cannot switch from windowed to windowless (and vice-versa) after instantiation. The m_drawable check should have been inside the windowed case to start with i.e as it is in the patch.
Simon Hausmann
Comment 4 2009-10-09 04:11:05 PDT
Comment on attachment 40876 [details] Create X Pixmap only when size changes
WebKit Commit Bot
Comment 5 2009-10-09 04:33:06 PDT
Comment on attachment 40876 [details] Create X Pixmap only when size changes Clearing flags on attachment: 40876 Committed r49386: <http://trac.webkit.org/changeset/49386>
WebKit Commit Bot
Comment 6 2009-10-09 04:33:10 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.