[Differential] [Request, 6 lines] D3190: [effects/translucency] Cancel existing animations before starting new

[graesslin (Martin Gräßlin)]

graesslin created this revision.
graesslin added reviewers: KWin, Plasma.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.

REVISION SUMMARY
  It can happen that startAnimation is invoked multiple times for a
  window. In case it was invoked a second time the previous animation was
  not cancelled. This resulted in the set-animation to never end. When
  closing a window, it would stay around as a translucent, non-interactive
  window zombie.
  
  This change ensures that existing animations get cancelled.
  
  BUG: 342716
  FIXED-IN: 5.8.3

TEST PLAN
  Tested through autotest and manually.

REPOSITORY
  rKWIN KWin

BRANCH
  effects-translucency-cancel-animations-5.8

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

AFFECTED FILES
  autotests/integration/effects/translucency_test.cpp
  effects/translucency/package/contents/code/main.js

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161028/6408c676/attachment.html>