Review Request 122140: Support reading IconPixmap and IconMask from WM_HINTS in NETWinInfo

Martin Gräßlin mgraesslin at kde.org
Mon Jan 19 11:03:09 UTC 2015


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

Review request for KDE Frameworks and kwin.


Repository: kwindowsystem


Description
-------

WM_HINTS property is already read and this provides the icon pixmap
and icon mask hint as read from WM_HINTS. This can be used for example
in KWindowSystem::icon which currently uses an XLib call.

Use NETWinInfo's WM2WindowClass and WM2IconPixmap in KWindowSystemPrivateX11::icon

Instead of using an XLib call the wrapper from NETWinInfo is used.
This reduces the number of roundtrips to the X server in case flags
includes both NETWM, ClassHint and WMHints.

In additon we don't need the deprecated x error eater any more.


Diffs
-----

  autotests/netwininfotestclient.cpp a0fdc403bb8329ea9fde786494bc0ccf88a8ebfd 
  src/kwindowsystem_x11.cpp bf958ae63b48424fc412405259f082b740928f9a 
  src/netwm.h 382ab460867905fdf0a4106a271e6614478fb438 
  src/netwm.cpp 1c96aaf9e480842c0b5379c89722f2e91a664476 
  src/netwm_def.h 49d02035eaec402a61e0b17b377b27c80d605b7b 
  src/netwm_p.h 0f1dd62ecd9c856e028dd33d28d461ed7099f60b 

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


Testing
-------


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150119/a72219ae/attachment.html>


More information about the Kde-frameworks-devel mailing list