Bug 30214

Summary: [Qt] Windowless Plugins : Create Pixmap only when size changes.
Product: WebKit Reporter: Girish Ramakrishnan <girish>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on: 20081    
Bug Blocks:    
Attachments:
Description Flags
Create X Pixmap only when size changes none

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.