kwin/autotests/integration/effects
Martin Flöser e1afef3d45 Sanity check WindowQuad before trying to create a grid out of it
Summary:
When one uses:
 * breeze as of 5.12
 * wobbly windows
 * shaded window
 * a distribution building with assert enabled

and starts to move a shaded window, KWin asserts. The root cause for
this is that WindowQuad::makeSubQuad has an assert for y1 being smaller
than y2. With the combination listed above this is not guaranteed. For
the left shadow quad the y1 and y2 are identical and thus trying to
split it, results in the assert condition.

The problem of the shadow quad having an invalid size might be addressed
as well with D10811. Due to that the generation of the quads is not
touched. Instead a sanity check is introduced to not try to split
already invalid sized quads.

BUG: 390953
FIXED-IN: 5.12.3

Test Plan: Added unit test hit the assert, now doesn't hit it any more

Reviewers: #kwin, #plasma

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D11015
2018-03-05 19:31:51 +01:00
..
CMakeLists.txt Sanity check WindowQuad before trying to create a grid out of it 2018-03-05 19:31:51 +01:00
fade_test.cpp XdgV6 - Kwin side 2017-09-25 16:47:30 +01:00
slidingpopups_test.cpp [autotests] Remove not needed includes for scene_qpainter.h 2017-09-01 17:44:49 +02:00
translucency_test.cpp [autotests] Remove not needed includes for scene_qpainter.h 2017-09-01 17:44:49 +02:00
windowgeometry_test.cpp [effects] Delay creation of EffectFrames in WindowGeometryEffect 2017-11-21 19:28:00 +01:00
wobbly_shade_test.cpp Sanity check WindowQuad before trying to create a grid out of it 2018-03-05 19:31:51 +01:00