D7259: [autotests] Remove not needed includes for scene_qpainter.h
Martin Flöser
noreply at phabricator.kde.org
Fri Aug 11 19:47:54 UTC 2017
graesslin created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Provide a virtual Scene::qpainterRenderBuffer() -> QImage* method
Needed by testing of QPainter scene to access the back buffer. Exposed
as a virtual method in Scene, so that the test does not have to cast to
SceneQPainter.
Move QPainter compositor into plugin
This change is similar to https://phabricator.kde.org/D7232 and moves the scene_qpainter into a
dedicated plugin. Compared to the XRender case it's more complicated as
the platform plugins need to implement a platform specific backend.
The base implementation for this part used to be in scene_qpainter. As
the idea is to completly move it away from KWin core it would be point
less to still have the backend definition in KWin core, but it cannot
be in the scene plugin as otherwise all platforms need to link the
plugin.
To solve this a new platformsupport subdirectory is added which contains
the scene platform backend as a static library. For the OpenGL scene such
a static library will also be required.
TEST PLAN
SceneQPainter test still passes, nested compositor still works
BRANCH
scene-qpainter-plugin
REVISION DETAIL
https://phabricator.kde.org/D7259
AFFECTED FILES
CMakeLists.txt
autotests/integration/effects/fade_test.cpp
autotests/integration/effects/slidingpopups_test.cpp
autotests/integration/effects/translucency_test.cpp
autotests/integration/scene_qpainter_test.cpp
autotests/integration/x11_client_test.cpp
composite.cpp
platformsupport/CMakeLists.txt
platformsupport/scenes/CMakeLists.txt
platformsupport/scenes/qpainter/CMakeLists.txt
platformsupport/scenes/qpainter/backend.cpp
platformsupport/scenes/qpainter/backend.h
plugins/platforms/drm/CMakeLists.txt
plugins/platforms/drm/scene_qpainter_drm_backend.h
plugins/platforms/fbdev/CMakeLists.txt
plugins/platforms/fbdev/scene_qpainter_fb_backend.h
plugins/platforms/virtual/CMakeLists.txt
plugins/platforms/virtual/scene_qpainter_virtual_backend.h
plugins/platforms/wayland/CMakeLists.txt
plugins/platforms/wayland/scene_qpainter_wayland_backend.h
plugins/platforms/x11/windowed/CMakeLists.txt
plugins/platforms/x11/windowed/scene_qpainter_x11_backend.h
plugins/scenes/CMakeLists.txt
plugins/scenes/qpainter/CMakeLists.txt
plugins/scenes/qpainter/qpainter.json
plugins/scenes/qpainter/scene_qpainter.cpp
plugins/scenes/qpainter/scene_qpainter.h
scene.cpp
scene.h
scene_qpainter.cpp
scene_qpainter.h
To: graesslin, #kwin, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170811/cc9299c1/attachment.html>
More information about the Plasma-devel
mailing list