Review Request 126640: [WindowThumbnail] Discard pixmap on map events.

Mihail Ivchenko ematirov at gmail.com
Tue Jan 5 15:29:05 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126640/
-----------------------------------------------------------

Review request for Plasma.


Repository: plasma-framework


Description
-------

According to [X.org docs](http://www.x.org/releases/X11R7.7/doc/compositeproto/compositeproto.txt) regarding NameWindowPixmap:

"This pixmap will remain allocated until freed, even if 'window' is unmapped, reconfigured or destroyed. However, 'window' will get a new pixmap allocated each time it is mapped or resized, so this request will need to be reinvoked"

So, pixmap needs to be discarded not only on XCB_CONFIGURE_NOTIFY event but on XCB_MAP_NOTIFY event also.


Diffs
-----

  src/declarativeimports/core/windowthumbnail.cpp 21e655e 

Diff: https://git.reviewboard.kde.org/r/126640/diff/


Testing
-------

Tested by using simple qml app with WindowThumbnail and minimizing\unminimizing of corresponding window. Now after unminimizing pixmap is still valid and live updating works.


Thanks,

Mihail Ivchenko

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160105/d2616e2e/attachment.html>


More information about the Plasma-devel mailing list