D7214: Add virtual Scene::scenePainter method

2017-08-09 Thread Martin Flöser
This revision was automatically updated to reflect the committed changes.
Closed by commit R108:ad4a3107d548: Add virtual Scene::scenePainter method 
(authored by graesslin).

REPOSITORY
  R108 KWin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D7214?vs=17914=17930

REVISION DETAIL
  https://phabricator.kde.org/D7214

AFFECTED FILES
  effects.cpp
  scene.cpp
  scene.h
  scene_qpainter.cpp
  scene_qpainter.h

To: graesslin, #kwin, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart, lukas


D7214: Add virtual Scene::scenePainter method

2017-08-09 Thread David Edmundson
davidedmundson accepted this revision.
This revision is now accepted and ready to land.

BRANCH
  scene-qpainter

REVISION DETAIL
  https://phabricator.kde.org/D7214

To: graesslin, #kwin, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart, lukas


D7214: Add virtual Scene::scenePainter method

2017-08-08 Thread Martin Flöser
graesslin created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  So far EffectsHandlerImpl directly accessed SceneQPainter::painter
  through a dynamic cast. If in future the QPainter based compositor should
  be moved into a plugin we cannot access it through a dynamic cast.
  
  To solve this problem the painter method is moved into Scene as
  a virtual method returning a sane default value.

BRANCH
  scene-qpainter

REVISION DETAIL
  https://phabricator.kde.org/D7214

AFFECTED FILES
  effects.cpp
  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